emulators/fceux: Update to 2.6.5 An interim release, gathering bug fixes and improvements from the past year. - move REINPLACE_CMD usage to pre-configure to make make makepatch easier to use. - commit 2d34c2b for PR #262886 has changed the package list to ship a man page we explicitly did not want to ship. Fix the post-install target and pkg-plist to continue not shipping the man page. Changelog: https://fceux.com/web/pressrelease-2.6.5.html
Details
- Reviewers
eduardo flo - Commits
- R11:09490acc3fef: emulators/fceux: Update to 2.6.5
Tested on i386 amd64 FreeBSD 12.4 13.1. arm64 pending.
See http://fuz.su/~fuz/freebsd/batch1/
Diff Detail
- Repository
- R11 FreeBSD ports repository
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Fixed the patch such that it actually picks version 2.6.5
instead of the earlier interim release. This required an
additional patch to fix a build issue.
See extended test logs at: http://fuz.su/~fuz/freebsd/batch1/fceux-2.6.5
- move REINPLACE_CMD usage to pre-configure to make make makepatch easier to use.
Thats a bad move to put reinplace cmds on pre-configure.
make makepatch should be run after make extract and changes made in WRKSRC to generate only wanted patches.
This doesn't make a whole lot of sense. If there are already existing patches, you need to run make patch to apply them, then edit them, and finally run make makepatch to generate new patches with your additional changes. If dynamic patching with REINPLACE_CMD is performed in post-patch, this workflow becomes impossible as we no longer get correct patches from make makepatch after having run make patch. And I'm certainly not going to apply all patches by hand every time I want to do a modification to one of them.
Hence the idea: Put REINPLACE_CMD into pre-configure where this issue is avoided with no changes to the build otherwise. As REINPLACE_CMD is for dynamic patching, i.e. patching in strings to configure it to our requirement, this fits better anyway. Many existing ports already do it this way, including audio/easytag, biology/bbmap, biology/bcftools, biology/bedtools, biology/bio-mocha, biology/bowtie2, biology/canu, biology/gemma, biology/haplohseq, biology/ncbi-entrez-direct, biology/pbbam, biology/py-ont-fast5-api, and biology/snpeff (just to name those from the biology category).
Fine with me. Don't see a problem using reinplace cmds on a different target than post-patch.