Bug 204130

Summary: [patch] astro/phoon: Update to 14Aug2014
Product: Ports & Packages Reporter: Torsten Zühlsdorff <ports>
Component: Individual Port(s)Assignee: Soeren Straarup <xride>
Status: Closed FIXED    
Severity: Affects Only Me CC: w.schwarzenfeld, xride
Priority: --- Keywords: patch
Version: LatestFlags: bugzilla: maintainer-feedback? (xride)
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch with update to 14Aug2014 ports: maintainer-approval? (xride)

Description Torsten Zühlsdorff 2015-10-29 16:58:56 UTC
Created attachment 162573 [details]
patch with update to 14Aug2014

Hello,

attached a patch to update astro/phoon to its current version.

portlint: WARN, missing license -> i scanned homepage and code, but could not find one
poudriere: OK for 9.3, 10.1 and 10.2 amd64
run time test: execute phoon and phoon with -l option - both works

Please note: i removed two patches because i could not figure out their need. Older PRs suggest that the patches should remove compiler warnings. But i did not see any difference in warnings with or without patches.

I've adjusted the Makefile patch to respect the actual CFLAGS. Because my C is not very good i've just assumed that the rest of the patch is okay and kept it.

To be sure i whould have created an Review but there is an long standing issue with umlauts in Diffs. Feel free to create a review yourself.

Greetings,
Torsten
Comment 1 Torsten Zühlsdorff 2015-11-04 15:18:01 UTC
I had done additional build tests. In summary the patched port builds fine on:

9.3 amd + i386
10.1 amd + i386
10.2 amd + i386
11-current r290334 amd + i386
Comment 2 Walter Schwarzenfeld 2018-01-09 01:51:40 UTC
ping
Comment 3 Tobias Kortkamp freebsd_committer freebsd_triage 2018-10-12 08:42:53 UTC
An update was committed in ports r471445...