Summary: | graphics/gif2png: Update to 2.5.11 | ||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Nathan <ndowens04> | ||||||||||||||
Component: | Individual Port(s) | Assignee: | Martin Wilke <miwi> | ||||||||||||||
Status: | Closed FIXED | ||||||||||||||||
Severity: | Affects Only Me | CC: | miwi, saper | ||||||||||||||
Priority: | --- | Flags: | saper:
maintainer-feedback+
|
||||||||||||||
Version: | Latest | ||||||||||||||||
Hardware: | Any | ||||||||||||||||
OS: | Any | ||||||||||||||||
URL: | http://www.catb.org/~esr/gif2png/NEWS | ||||||||||||||||
Attachments: |
|
Description
Nathan
2018-08-15 22:35:54 UTC
Created attachment 196240 [details]
Update
graphics/gif2png:
* Removed localbase from USES
-- Does not affect current version
* Removed LICENSE_FILES
-- LICENSE file no longer exists
* Updated port to use source's Makefile
-- I prefer to use included scripts
unless it's impossible to use
Portlint -AC: OK
poudriere(amd64): 10.4, 11.2 worked fine
poudriere(mips): 11.2 Fine
poudriere(armv6): Fine
poudriere(arm64): Fine
Created attachment 196241 [details]
Update
Thanks for picking this up! I get this: # make check-plist ====> Checking for pkg-plist issues (check-plist) ===> Parsing plist ===> Checking for items in STAGEDIR missing from pkg-plist Error: Orphaned: share/man/man1/gif2png.1 Error: Orphaned: share/man/man1/web2png.1 ===> Checking for items in pkg-plist which are not in STAGEDIR Error: Missing: share/man/man1/gif2png.1.gz Error: Missing: share/man/man1/web2png.1.gz (In reply to Marcin Cieślak from comment #3) That is weird as I started from clear port dir, applied patch and with svn patch and use poudriere for both armv6 and amd64 and I get no errors Probably something with my environment (this was on a 10.4 box), but still something that may happen to some... (In reply to Marcin Cieślak from comment #5) Doesn't do it on poudriere or actually on my host box. Nor on poudriere(mips) Created attachment 196804 [details]
Update gif2png
This is my patch. It fixes manpage location and after this "make check-plist" completes without an error.
Created attachment 196807 [details] Update; fix man location graphics/gif2png: * Removed localbase from USES -- Does not affect current version * Removed LICENSE_FILES -- LICENSE file no longer exists * Updated port to use source's Makefile -- I prefer to use included scripts unless it's impossible to use * Replaced cp in Makefile to strip binaries -- Avoid having to manually strip it Changelog:http://www.catb.org/~esr/gif2png/NEWS Thanks, it is much cleaner now... but I feel I am getting this build error with your diff: --- gif2png.o --- gif2png.c:742:2: warning: add explicit braces to avoid dangling else [-Wdangling-else] else ^ --- memory.o --- cc -O2 -pipe -fstack-protector -fno-strict-aliasing -I/usr/local/include -c memory.c -o memory.o --- version.o --- cc -O2 -pipe -fstack-protector -fno-strict-aliasing -I/usr/local/include -c version.c -o version.o version.c:17:34: error: expected ';' after top level declarator const char version[] = "gif2png " VERSION; ^ ; 1 error generated. *** [version.o] Error code 1 Created attachment 197648 [details] Update graphics/gif2png: * Removed localbase from USES -- Does not affect current version * Removed LICENSE_FILES -- LICENSE file no longer exists * Updated port to use source's Makefile -- I prefer to use included scripts unless it's impossible to use Changes: http://www.catb.org/~esr/gif2png/NEWS PR: 230660 Submitted by: Nathan <ndowens@yahoo.com> Thanks, that's not a kind of fix I would go for. Need to investigate this a bit. A commit references this bug: Author: miwi Date: Fri Jan 11 14:51:58 UTC 2019 New revision: 489975 URL: https://svnweb.freebsd.org/changeset/ports/489975 Log: - Update to 2.5.11 PR: 230660 Submitted by: ndowens@yahoo.com Approved by: maintainer timeout Sponsored by: iXsystems Inc. Changes: head/graphics/gif2png/Makefile head/graphics/gif2png/distinfo head/graphics/gif2png/files/ head/graphics/gif2png/files/patch-Makefile |