#18662 closed update (fixed)
Update for awstats Portfile to 6.9
| Reported by: | davis@… | Owned by: | alakazam@… |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | 1.7.0 |
| Keywords: | Cc: | davis@… | |
| Port: | awstats |
Description
I could take over as maintainer as this is currently nomaintainer. Let me know.
Attachments (1)
Change History (11)
comment:1 follow-up: 2 Changed 16 years ago by alakazam@…
| Owner: | changed from macports-tickets@… to alakazam@… |
|---|---|
| Status: | new → assigned |
comment:2 follow-up: 4 Changed 16 years ago by davis@…
comment:3 Changed 16 years ago by alakazam@…
Committed in r47359. I've taken you up on your maintainership offer as well and assigned maintainership of awstats to you.
comment:4 follow-up: 6 Changed 16 years ago by alakazam@…
Changed 16 years ago by davis@…
| Attachment: | awstats_port.diff added |
|---|
comment:5 Changed 16 years ago by alakazam@…
| Milestone: | Port Enhancements → Port Updates |
|---|---|
| Resolution: | → fixed |
| Status: | assigned → closed |
Ok I guess I didn't reload the diff correctly ; fixed in r47360.
comment:6 follow-up: 8 Changed 16 years ago by davis@…
Replying to alakazam@…:
Replying to davis@…:
I'm not sure what changed between your two patches, could you review r47359 and tell me if I made a mistake ? Thanks !
Sorry, I should have taken more time before posting the diff. I forgot to ensure that all the perl scripts that have been added between 6.5 and 6.9 have all their /usr/local and /usr/bin/perl replaced. I think I have it correct now (works on my system), plus I included your addition of me as maintainer. Thanks.
Does that mean my maintainer name is "guydavis.ca:davis"? Do I need to commit directly to SVN now? Sorry, I'm new to port admin. I probably will be taking over rb-rmagick from febeling@… after we sort out a current problem. I hope to help with more ports as time goes on.
Thanks again, Guy
comment:7 Changed 16 years ago by alakazam@…
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
comment:8 Changed 16 years ago by alakazam@…
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
Replying to davis@…:
Sorry, I should have taken more time before posting the diff. I forgot to ensure that all the perl scripts that have been added between 6.5 and 6.9 have all their /usr/local and /usr/bin/perl replaced. I think I have it correct now (works on my system), plus I included your addition of me as maintainer. Thanks.
No problem. I fixed the remaining occurrences of /usr/bin/ in r47362 as per your new diff.
Does that mean my maintainer name is "guydavis.ca:davis"?
This reverse notation is actually there to obfuscate your email address but as you can see with port info awstats, Macports knows how to decrypt it when querying information about a port's maintainer.
Do I need to commit directly to SVN now? Sorry, I'm new to port admin.
Actually maintaining ports and having commit access are two different things ; people who maintain several ports on a regular basis often ask for commit rights in order to commit updates more easily. You can find out more about commit access in the guide.
I hope to help with more ports as time goes on.
That's really nice !
comment:9 Changed 16 years ago by jmroot (Joshua Root)
| Type: | enhancement → update |
|---|
Replying to alakazam@…:
Hi, I missed the new awstats_configure.pl script to ensure that it's /usr/local are replaced. Added it in my second upload of the diff.