Created attachment 150018 [details] sysutils/cdroot MASTER_SITES, PORTVERSION, req MAINTAINER sysutils/cdroot changes PORTVERSION MASTER_SITES MAINTAINER modifications (minor) to source, which necessitated the (port) version bump. requests MAINTAINER Please see svn(1) diff(1), attached. Thanks! --Chris
Created attachment 150050 [details] sysutils/cdroot MASTER_SITES, PORTVERSION, req MAINTAINER Obloletes previous patch (svn(1) diff(1)). Thanks! --Chris
This is a version upgrade. I pretty much don't promote any PRs that update versions without seeing logs. Somebody else might (they haven't yet in 2 months), so you may want to attach a real poudriere log to improve this PR's changes. :)
Created attachment 151966 [details] sysutils/cdroot MASTER_SITES, PORTVERSION, req MAINTAINER (svn diff) pet portlint: reduce COMMENT add LICENSE Please also find cdroot-QA (QA session) attached. Thanks, John. As always; good call. --Chris
Created attachment 151967 [details] QA session, as per John's request
A commit references this bug: Author: riggs Date: Sat Feb 21 07:29:51 UTC 2015 New revision: 379481 URL: https://svnweb.freebsd.org/changeset/ports/379481 Log: - Update to upstream version 1.2.6 - New maintainer: Chris Hutchinson <portmaster@bsdforge.com> PR: 195520 Submitted by: portmaster@bsdforge.com (maintainer) Reviewed by: koobs, riggs Changes: head/sysutils/cdroot/Makefile head/sysutils/cdroot/distinfo head/sysutils/cdroot/files/ head/sysutils/cdroot/pkg-descr
Thanks for taking maintainership! Committed. Just a brief note for future submissions: Your diff did not indicate the (necessary, because it is included in the 1.2.6 release already) removal of files/patch-Makefile. Probably you did not run 'svn rm' on them.
(In reply to Thomas Zander from comment #6) > Thanks for taking maintainership! Committed. > > Just a brief note for future submissions: Your diff did not indicate the > (necessary, because it is included in the 1.2.6 release already) removal of > files/patch-Makefile. Probably you did not run 'svn rm' on them. Thank *you*, Thomas. Sorry about the svn diff. I *usually* remember to svn delete/add as necessary, prior to creating the diff. But apparently forgot. :( Thanks again! --Chris