Bug 225867

Summary: [PATCH][ADOPT] games/xroach: fix reason for being BROKEN and new maintainer
Product: Ports & Packages Reporter: Philipp Engel <kidon>
Component: Individual Port(s)Assignee: Romain Tartière <romain>
Status: Closed FIXED    
Severity: Affects Only Me CC: romain
Priority: --- Keywords: patch
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Patch to update the source code.
none
Patch to update the Makefile and distinfo.
none
New distinfo.
none
svn patch
none
new version none

Description Philipp Engel 2018-02-13 17:44:22 UTC
Created attachment 190583 [details]
Patch to update the source code.
Comment 1 Philipp Engel 2018-02-13 17:45:05 UTC
Created attachment 190584 [details]
Patch to update the Makefile and distinfo.
Comment 2 Philipp Engel 2018-02-13 17:51:28 UTC
Created attachment 190586 [details]
New distinfo.
Comment 3 Romain Tartière freebsd_committer freebsd_triage 2018-02-13 20:39:29 UTC
Hi Philipp,

Can you please update this PR to provide a single patch conforming to the recommendations of the FreeBSD Porter's Handbook?

https://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/book.html#svn-diff

Thanks!
Comment 4 Philipp Engel 2018-02-13 20:57:23 UTC
Created attachment 190594 [details]
svn patch

Hi Romain, hope that I've got it right.
Comment 5 Philipp Engel 2018-02-13 21:01:47 UTC
Created attachment 190595 [details]
new version

Mercy me, I've forgotten the source code patch.
Comment 6 Romain Tartière freebsd_committer freebsd_triage 2018-02-13 21:04:32 UTC
Excellent!  I'll give a look tomorrow
Comment 7 commit-hook freebsd_committer freebsd_triage 2018-02-14 08:09:43 UTC
A commit references this bug:

Author: romain
Date: Wed Feb 14 08:08:49 UTC 2018
New revision: 461766
URL: https://svnweb.freebsd.org/changeset/ports/461766

Log:
  Resurrect, unbreak and modernize games/xroach

  Submitter is also taking maintainership.

  PR:		225867
  Submitted by:	kidon@posteo.de

Changes:
  head/games/xroach/
  head/games/xroach/Makefile
  head/games/xroach/distinfo
  head/games/xroach/files/patch-Makefile
  head/games/xroach/files/patch-xroach.c
Comment 8 Romain Tartière freebsd_committer freebsd_triage 2018-02-14 08:12:42 UTC
Thank you for resurrecting these cockroaches and taking maintainership!

A few notes for your future submissions:

- ports-mgmt/portlint can report a variety of problems, it's a good idea to ensure it does not report errors;
- You can set DEVELOPER=yes in /etc/make.conf so that more sanity checks are performed when you update ports.

I fixed the problems reported by these tools.  If you do not understand a change I made, feel free to ask me what happened ;-)