Created attachment 146625 [details] net/balance [maintainer] STAGE pkg-plist net/balance adds MAINTAINER, additional entry to pkg-plist Trivial update. All total; 3 lines. Please see svn(1) diff(1), for details. --Chris P.S. This is the last [new] one I'll submit, until I get poudriere working properly. Sorry for any [previous] difficulties, and thanks for all your help, and understanding.
<grumble> seriously, please stop putting "[maintainer]" in the title when you aren't already the maintainer. This is the second time I've asked.
And rejected, it contains "MAN1=" I have to believe you don't understand why "MAN<X>=" in staging PR is wrong. I've pointed this out about 5 times and you keep doing it. The only reasonable explanation is you don't know how to handle staging. you aren't following this: https://wiki.freebsd.org/ports/StageDir I'm rejecting this PR because it's again untested and this is grossly wrong. It didn't even contain the minimum of "make check-plist", "make stage-qa", portlint, and "make check-sanity" which I have requested numerous times. These are REQUIREMENTS for your that I won't waive.
(In reply to John Marino from comment #1) > <grumble> > seriously, please stop putting "[maintainer]" in the title when you aren't > already the maintainer. This is the second time I've asked. Sorry. This is what I have always used in the past, given it is maintainer related (request for maintainer). I thought it pertinent. Would you prefer [maintainer request]? or ??? --Chris
No. I prefer "and request maintainership" in the title. I've changed your titles quite a few times and those changes serve as examples. I assume you got notices showing the title change. So ideally you would notice these changes which show 1) what not to do and 2) what to do instead. but in this case I explicitly pointed it out. "[maintainer]" means the current maintainer opened the PR, not somebody that wants to be a maintainer. It's a way of establishing credibility quickly.
(In reply to John Marino from comment #4) > No. I prefer "and request maintainership" in the title. > > I've changed your titles quite a few times and those changes serve as > examples. I assume you got notices showing the title change. So ideally > you would notice these changes which show 1) what not to do and 2) what to > do instead. > > but in this case I explicitly pointed it out. > > > > "[maintainer]" means the current maintainer opened the PR, not somebody that > wants to be a maintainer. It's a way of establishing credibility quickly. Sorry. I _apparently_ overlooked (missed) the title change. Point _well_ taken, and acknowledged. :) --Chris
Created attachment 146755 [details] net/balance [request for maintainer] STAGE LICENSE COMMENT pkg-plist svn diff (test log) net/balance request for maintainer adds STAGEDIR MAINTAINER changes Makefile pkg-plist This is a request for MAINTAINERship. I have added STAGEDIR to this, and have made additional modifications to ensure that this port meets current requirements. Please see 2014-09-03.diff, and net-balance-tests-out, for details. Thank you for all your time, and consideration. --Chris
Created attachment 146756 [details] net/balance [request for maintainer] output log for 2014-09-03.diff (net-balance-tests-out) net/balance Requisite testing logs for 2014-09-03.diff, also attached. Thank you for all your time, and consideration. --Chris
Needs Triage