Created attachment 163882 [details] Fix preprocessor stringification Freesweep segfaults when you win a game. The cause is mishandled stringification in the source. I've attached two patch files for the port that resolve the issue. This properly belongs upstream, but freesweep looks unmaintained and the contact address in the README is no longer valid. Not sure what to do about that.
Thanks for your submission, Randy! Can you please attach a patch in unified diff format, and if possible confirm that it passes QA (portlint, poudriere)?
Created attachment 163913 [details] change upstream and update version Rather than patching this locally, I forked the abandoned upstream and fixed the segfault there. The last commit was 2003, so I'm adopting it. The attached patch changes the upstream repo, bumps PORTVERSION from 0.92 to 0.93, and sets me as the maintainer. It passes portlint and builds successfully.
Thanks for your submission. Since you're essentially forking the upstream, I'd like to ask you to file a new bug report that is a new port request with a shar file. It'd be best if you renamed it as well.
There's absolutely no need to send shar or rename the project. Apart from minor shortcomings this looks good to me.
Leave this on hold for the moment. I've finally managed to get in contact with the original authors and am now officially a co-maintainer. There will be a 1.0 release with the fix very soon. I'll update the patch as soon as that happens.
Ok. I'll attach my version of the patch.
Created attachment 164814 [details] Corrected patch - PORTREVISION shold be removed when version is updated - No need for MASTER_SITES with USE_GITHUB - DISTVERSIONPREFIX is preferred to GH_TAGNAME - GH_PROJECT not needed, same as PORTNAME - While here, convert to conditional docs target Just noticed that it lists LICENSE=GPLv2 GPLv3 / LICENSE_COMB=dual, which is incorrect for GPLv2+. It should be "LICENSE=GPLv2 # or later" (yes, with comment) for now
Hi, Any progress here?
I'm still waiting on the original author to check off on my changes. I'll ping him again and update this as soon as I hear back.
Created attachment 165797 [details] update to 1.0 New 1.0 patch uploaded, with Dmitry's fixes included. This is finally ready to go.
(In reply to Randy Westlund from comment #10) It's actually 'LICENSE= GPLv2+' now as support for the 'or later' has been added to the tree. However I notice the internal version does not match the PORTVERSION... cc -Wall -DVERSION=\"0.93\" -O2 -pipe -fstack-protector -fno-strict-aliasing -c clear.c
Created attachment 167001 [details] update to 1.0.1 Good catch, thanks. Fixed.
A commit references this bug: Author: junovitch Date: Sun Feb 14 22:22:40 UTC 2016 New revision: 408903 URL: https://svnweb.freebsd.org/changeset/ports/408903 Log: games/freesweep: update 0.92 -> 1.0.1 - Transfer MAINTAINER from ports@ to submitter - Update LICENSE - Switch from GOOGLE_CODE to USE_GITHUB; use the maintainer's repository - While here, convert to conditional docs target - Expand on pkg-descr - MFH as the original upstream version seg faults upon winning a game and the maintainer is now a co-maintainer with the upstream project PR: 205052 Submitted by: Randy Westlund <rwestlun@gmail.com> (incoming maintainer) Reviewed by: amdmi3 (earlier version) MFH: 2016Q1 Changes: head/games/freesweep/Makefile head/games/freesweep/distinfo head/games/freesweep/pkg-descr
I've requested MFH as the version in quarterly would not be usuable if it segfaults and it's a non-invasive update. Take and set merge-quarterly? until I get confirmation on the MFH approval.
A commit references this bug: Author: junovitch Date: Sun Feb 14 22:35:28 UTC 2016 New revision: 408907 URL: https://svnweb.freebsd.org/changeset/ports/408907 Log: MFH: r408903 games/freesweep: update 0.92 -> 1.0.1 - Transfer MAINTAINER from ports@ to submitter - Update LICENSE - Switch from GOOGLE_CODE to USE_GITHUB; use the maintainer's repository - While here, convert to conditional docs target - Expand on pkg-descr - MFH as the original upstream version seg faults upon winning a game and the maintainer is now a co-maintainer with the upstream project PR: 205052 Submitted by: Randy Westlund <rwestlun@gmail.com> (incoming maintainer) Reviewed by: amdmi3 (earlier version) Approved by: ports-secteam (feld) Changes: _U branches/2016Q1/ branches/2016Q1/games/freesweep/Makefile branches/2016Q1/games/freesweep/distinfo branches/2016Q1/games/freesweep/pkg-descr
Close. Thanks!