Bug 220973 - net/gopher: Update to 3.0.6, Take MAINTAINER'ship, portlint cleanup
Summary: net/gopher: Update to 3.0.6, Take MAINTAINER'ship, portlint cleanup
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Richard Gallamore
URL:
Keywords: needs-patch
Depends on:
Blocks:
 
Reported: 2017-07-24 18:06 UTC by Aaron Baugher
Modified: 2017-08-12 04:57 UTC (History)
1 user (show)

See Also:


Attachments
'svn diff' of net/gopher changes. (3.99 KB, patch)
2017-07-24 18:06 UTC, Aaron Baugher
no flags Details | Diff
Poudriere testport log for net/gopher on 11.0 amd64 (31.71 KB, text/x-log)
2017-07-24 18:10 UTC, Aaron Baugher
no flags Details
Poudriere testport log for net/gopher on 11.0 i386 (31.57 KB, text/x-log)
2017-07-24 18:10 UTC, Aaron Baugher
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Aaron Baugher 2017-07-24 18:06:39 UTC
Created attachment 184668 [details]
'svn diff' of net/gopher changes.

I'd like to be the maintainer for this port, because I miss gopher from the old days and I'm working on a gopher server, so I use this client sometimes.

I bumped PORTREVISION because adding the LICENSE to the Makefile added files to the resulting package.  I also cleaned up several portlint errors and warnings, including turning the ports into 'make makepatch' ports, reordering the lines in the Makefile, and adding the license.

Tested in poudriere, will attach logs.
Comment 1 Aaron Baugher 2017-07-24 18:10:02 UTC
Created attachment 184669 [details]
Poudriere testport log for net/gopher on 11.0 amd64
Comment 2 Aaron Baugher 2017-07-24 18:10:34 UTC
Created attachment 184670 [details]
Poudriere testport log for net/gopher on 11.0 i386
Comment 3 Kubilay Kocak freebsd_committer freebsd_triage 2017-07-25 04:24:16 UTC
Thank you for offering to take maintainership of a port Aaron

Note for future issues that attaching QA logs is not necessary unless requested, but please continue to confirm what QA has been run and their results, something like:

portlint: OK (looks fine.)
testport: OK (poudriere: <versions>, <archs>, <options> tested)
maketest: OK (XXX passed, 0 failed)
Comment 4 Aaron Baugher 2017-07-25 13:28:11 UTC
(In reply to Kubilay Kocak from comment #3)
Thanks, I'll do that.
Comment 5 Richard Gallamore freebsd_committer freebsd_triage 2017-07-31 17:15:49 UTC
(In reply to Aaron Baugher from comment #4)
Does the source come with a license file? Is adding LICENSE_FILE possible?
Comment 6 Aaron Baugher 2017-08-01 12:40:46 UTC
(In reply to Richard Gallamore from comment #5)
Yes, it has one in ${WRKSRC}/COPYING.

Should I add that and attach a revised diff here?  Thank you.
Comment 7 Kubilay Kocak freebsd_committer freebsd_triage 2017-08-01 12:48:08 UTC
Yes please Aaron :)
Comment 8 commit-hook freebsd_committer freebsd_triage 2017-08-08 22:15:49 UTC
A commit references this bug:

Author: ultima
Date: Tue Aug  8 22:15:25 UTC 2017
New revision: 447579
URL: https://svnweb.freebsd.org/changeset/ports/447579

Log:
  * Give maintainer to submitter, Aaron Baugher
  * Added license information
  * Cleaned up Makefile

  PR:		220973
  Submitted by:	Aaron Baugher (maintainer)
  Reviewed by:	matthew (mentor), mat
  Approved by:	matthew (mentor)
  Differential Revision:	https://reviews.freebsd.org/D11897

Changes:
  head/net/gopher/Makefile
  head/net/gopher/files/patch-gopher_form.c
  head/net/gopher/files/patch-object_BLblock.c
  head/net/gopher/files/patch-object_GSgopherobj.c
  head/net/gopher/files/patch-object_Regex.c
  head/net/gopher/files/patch-object_url.c
Comment 9 Richard Gallamore freebsd_committer freebsd_triage 2017-08-09 00:57:06 UTC
Added the license file and committed, thanks.
Comment 10 Aaron Baugher 2017-08-12 04:57:16 UTC
Thank you.  I was trying to create a new diff with the LICENSE_FILE added, and couldn't figure out why the diff was empty, and it was because you had already made the change!  Thanks again.