Created attachment 171109 [details] Proposed patch (since 416324 revision) Patch to unbreak and modernize games/imaze port. - Bump PORTREVISION - Unbreak fetch - Add ATHENA option and set it by default, instead of XVIEW - Convert GUI's options from RADIO to SINGLE - Switch to options helpers - Add MOTIFLIB define with proper value to fix linkage - Add xmu to USE_XORG - Regenerate patches The proposed ATHENA option allows to differentiate between MOTIF and XVIEW options (while they also use Xaw library), because they define TOOLKIT variable, with the same meaning as the previous approach. The ATHENA option defined by default, because it allows to build on 64-bit platforms, unlike the XVIEW option. Actually, the port uses files/patch-source_Makefile.in patch, which replaces "MotifLibs=-lXm -lXt" with "MotifLibs=${MOTIFLIB}", but USES+=motif defines MOTFILIB?="-L${LOCALBASE}/lib -lXm -lXp", where "-lXt" needed instead of "-lXp". Therefore, either value of MOTFILIB could be fixed for Mk/Uses/motif.mk file, or I just defined it to proper value with "-lXt". The distfile is fetchable: % curl -sLI http://www.sourcefiles.org/Games/Arcade/imaze-1.4.tar.gz HTTP/1.1 200 OK Date: Mon, 06 Jun 2016 16:14:00 GMT Server: Apache/2.2.3 (Fedora) Last-Modified: Sun, 18 Mar 2007 02:00:00 GMT ETag: "920ab-9e9e9-d4898800" Accept-Ranges: bytes Content-Length: 649705 Connection: close Content-Type: application/x-gzip
Created attachment 171110 [details] The poudriere testport log (FreeBSD 10.2 amd64, with default options)
Created attachment 171111 [details] The poudriere testport log (FreeBSD 10.2 amd64, with MOTIF=on)
Created attachment 171112 [details] The poudriere testport log (FreeBSD 10.2 i386, with XVIEW=on)
Possible to move MOTIFLIB define to global space. I configured it for MOTIF option, but this is not mandatory.
Cleaning up summary in favor of keywords.
Comment on attachment 171109 [details] Proposed patch (since 416324 revision) Implicit approval, the port has no maintainer.
Hello lightside, thanks for the patch! Since the patches did not change at all, can you please provide a diff without them? I started the testbuilds a moment ago without the patches and will let them run over night. Greetings, Torsten
Created attachment 171394 [details] Proposed patch (since 416324 revision) without regenerated patches Hello Torsten Zuehlsdorff. (In reply to comment #7) > Since the patches did not change at all, can you please provide a diff > without them? I can, but these patches was regenerated with `make makepatch` command to fix portlint warnings, like in following kind of changes: ports r416708, ports r416584, ports r416495, etc.
Hello, since i'm new to the ports-team i asked my mentors how to handle the patches. Conclusion: regeneration of the patches for petting portlint is just fine. No need to change the patch. I will process them as they are! Thanks, Torsten
Comment on attachment 171394 [details] Proposed patch (since 416324 revision) without regenerated patches (In reply to comment #9) > Conclusion: regeneration of the patches for petting portlint is just fine. > No need to change the patch. I will process them as they are! Ok, thanks.
A commit references this bug: Author: tz Date: Fri Jun 17 08:21:17 UTC 2016 New revision: 417000 URL: https://svnweb.freebsd.org/changeset/ports/417000 Log: games/imaze: unbreak and modernize port This includes: - Bump PORTREVISION - Unbreak fetch - Remove TUCOWS from MASTER_SITES - Add ATHENA option and set it by default, instead of XVIEW - Convert GUI's options from RADIO to SINGLE - Switch to options helpers - Add MOTIFLIB define with proper value to fix linkage - Add xmu to USE_XORG - Regenerate patches PR: 210081 Submitted by: lightside <lightside@gmx.com> Approved by: junovitch (mentor) Changes: head/games/imaze/Makefile head/games/imaze/files/patch-source_Makefile.in head/games/imaze/files/patch-source_configure head/games/imaze/files/patch-source_fbsd_joystick.c head/games/imaze/files/patch-source_voxware_audio.c
Committed, thanks! :) As sidenote: i also removed TUCOW from the master-sites, since the source was not fetchable from it.