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:
Fix category (submitter is not maintainer) (via the GNATS Auto Assign
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:
Edwin Groothuis via the GNATS Auto Assign Tool
Awaiting maintainers feedback (via the GNATS Auto Assign Tool)
I'd call this a maintainer timeout.
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.
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.
heh, I got the email and I thought it came from bugzilla. Okay, so there's your answer although the question is missing. :)
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.
okay, we'll make sure you get listed as the new maintainer.
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.
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.
It looks like you overlooked that name change:
^^^ that was on rss2email2 as you proposed it.
A commit references this bug:
Date: Tue Aug 19 11:52:30 UTC 2014
New revision: 365371
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
Submitted by: Stephen R. Guglielmo
Tweaked by: marino
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.
hmm, rssemail3 pkgname is also rss2email so I probably shouldn't change it.
A commit references this bug:
Date: Tue Aug 19 12:26:50 UTC 2014
New revision: 365373
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.
Submitted by: Stephen Guglielmo
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.