|Summary:||[Patch] Rename and Update of mail/rss2email Ports|
|Product:||Ports & Packages||Reporter:||Steve <srg>|
|Component:||Individual Port(s)||Assignee:||John Marino <marino>|
|Severity:||Affects Only Me||CC:||cs, marino, srg|
Description Steve 2014-05-29 20:50:02 UTC
This is a patch for a few things: -Update the conflicts for mail/rss2email3 -Update mail/rss2email2 and take over maintenance -Rename mail/rss2email to mail/rss2email2 to avoid ambiguity -Add the entry to MOVED and fix some minor LINT issues Fix: Patch attached with submission follows:
Comment 1 Edwin Groothuis 2014-05-29 20:50:07 UTC
Class Changed From-To: maintainer-update->change-request Fix category (submitter is not maintainer) (via the GNATS Auto Assign Tool)
Comment 2 Edwin Groothuis 2014-05-29 20:50:08 UTC
Maintainer of mail/rss2email, Please note that PR ports/190381 has just been submitted. If it contains a patch for an upgrade, an enhancement or a bug fix you agree on, reply to this email stating that you approve the patch and a committer will take care of it. The full text of the PR can be found at: http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/190381 -- Edwin Groothuis via the GNATS Auto Assign Tool edwin@FreeBSD.org
Comment 3 Edwin Groothuis 2014-05-29 20:50:09 UTC
State Changed From-To: open->feedback Awaiting maintainers feedback (via the GNATS Auto Assign Tool)
Comment 4 Carlo Strub 2014-08-10 21:56:46 UTC
I'd call this a maintainer timeout.
Comment 5 John Marino 2014-08-11 08:47:05 UTC
Carlos, feel free to move these to "patch-ready" status with a note that anybody can commit. The implication that testing the patch for build issues will fall to the committer.
Comment 6 John Marino 2014-08-11 15:44:38 UTC
I'm probably going to propose a workflow soon (flow chart) based on my recent experiences. I've been using "patch-ready" to mean "the changes described in the PR (patches, text, what) have been evaluated to some extent, and should reasonably apply and pass all build tests". Basically it's highest status a PR can achieve prior to be committed, so the person setting it is stating they think the PR is ready for committing. This is why I've increasing been insisting on build logs. I give maintainer updates more room, especially if it's a known maintainer with a good reputation and it's not a staging PR.
Comment 7 John Marino 2014-08-11 15:45:34 UTC
heh, I got the email and I thought it came from bugzilla. Okay, so there's your answer although the question is missing. :)
Comment 8 Steve 2014-08-16 15:55:03 UTC
I'm happy to take over maintaining the port (which is why I included it in my update). I attempted to contact the maintainer prior to submitting this PR, but I got no response.
Comment 9 John Marino 2014-08-16 18:16:19 UTC
okay, we'll make sure you get listed as the new maintainer.
Comment 10 John Marino 2014-08-19 10:55:28 UTC
In the future, do not make changes to files that don't pertain to your PR, e.g. -sysutils/cpupowerd||2014-05-04|Has expired: Not maintained from upstream anymore and only supports ancient AMD K8. +sysutils/cpupowerd||2014-05-04|Has expired: Not maintained from upstream anymore and only supports ancient AMD K8 net/pchar||2014-05-04|Has expired: Obsolete, abandoned x11-fonts/texcm-ttf||2014-05-04|Has expired: Should be replaced by x11-fonts/stix-fonts devel/buildapp||2014-05-05|Has expired: Broken for more than 6 months converters/konwert||2014-05-05|Broken with modern compilers, not staged, no upstream -graphics/gimp-gap||2014-05-05|Broken with gimp 2.8, no release in 5 years. -games/gnome-games-extra-data||2014-05-05|Not used in GNOME 3. +graphics/gimp-gap||2014-05-05|Broken with gimp 2.8, no release in 5 years +games/gnome-games-extra-data||2014-05-05|Not used in GNOME 3 don't do this. I doubt it even applies anymore.
Comment 11 John Marino 2014-08-19 11:08:34 UTC
I think I'm going to do this in two steps (the diff isn't helpful here but I'll manage). step 1) update mail/rss2email including maintainer change step 2) svn mv + portname change + MOVED change. That will probably preserve history better.
Comment 12 John Marino 2014-08-19 11:12:52 UTC
It looks like you overlooked that name change: PORTNAME= rss2email ^^^ that was on rss2email2 as you proposed it.
Comment 13 commit-hook 2014-08-19 11:53:23 UTC
A commit references this bug: Author: marino Date: Tue Aug 19 11:52:30 UTC 2014 New revision: 365371 URL: http://svnweb.freebsd.org/changeset/ports/365371 Log: www/rss2email: Upgrade version 2.70 => 2.71 and assign maintainer This is the first part of a two-part PR, and it involves upgrading rss2email. The second part will relocate the port. During the first part, the following changed: * python prefix added to package name * USE_PYTHON converted to USES=python * Upgraded to version 2.71 * Assign maintainership to submitter * Update the comment * Note GPLv2 and GPLv3 dual licensing * Improve RUN_DEPENDS specification * Note conflict with py-rss2email3 PR: 190381 Submitted by: Stephen R. Guglielmo Tweaked by: marino Changes: head/mail/rss2email/Makefile head/mail/rss2email/distinfo head/mail/rss2email/pkg-descr
Comment 14 John Marino 2014-08-19 11:54:56 UTC
Note that your update had a couple of bugs in it. 1) You removed NO_BUILD, which caused it to fail looking for a makefile 2) You added some DOCS stuff but never installed it. I just removed the docs stuff from the plist for now. That causes a stage-qa failure. In it's form now, poudriere is happy.
Comment 15 John Marino 2014-08-19 12:18:14 UTC
hmm, rssemail3 pkgname is also rss2email so I probably shouldn't change it.
Comment 16 commit-hook 2014-08-19 12:27:25 UTC
A commit references this bug: Author: marino Date: Tue Aug 19 12:26:50 UTC 2014 New revision: 365373 URL: http://svnweb.freebsd.org/changeset/ports/365373 Log: Move mail/rss2email => mail/rss2email2 This is done to avoid confusion with mail/rss2email3. Also update the conflicts line of rss2email3 that broke with the recent update of mail/rss2email. This is the second part of the PR. PR: 190381 Submitted by: Stephen Guglielmo Changes: head/MOVED head/mail/Makefile head/mail/rss2email/ head/mail/rss2email2/ head/mail/rss2email2/Makefile head/mail/rss2email3/Makefile
Comment 17 John Marino 2014-08-19 12:29:08 UTC
Comment 18 Steve 2014-08-21 14:05:43 UTC
Re: Changing other files. I did this because I noticed some LINT issues with the MOVED file as I was adding my own entry for this PR. I'm not super familiar with FreeBSD's policies/practices, but I understand more now. Sorry for including those changes in the PR! Thanks for closing this, and I'll review your changes in the commits to see what you did and what I did wrong.