Differential D55509 Authored by royger on Feb 25 2026, 2:19 PM. Tags None Referenced Files
Subscribers None
Details Summary GNU Binutils packages strip, and should be used in preference to the base Do not take over the STRIP makefile variable, which is already used in bsd.port.mk to signal whether the install program should strip the binaries. Instead define STRIP_CMD and use STRIP just for the environment variables.
Diff Detail
Event Timelineroyger requested review of this revision.Feb 25 2026, 2:19 PM royger created this revision. dim accepted this revision.Feb 25 2026, 3:25 PM Comment ActionsSounds reasonable to me, but I'm not a ports committer. :) This revision is now accepted and ready to land.Feb 25 2026, 3:25 PM royger added a comment.Feb 25 2026, 3:34 PM Comment ActionsPicked the last couple of people that modified that file, and you happened to be there :). Thanks, will wait a bit for someone from ports to voice an opinion. arrowd accepted this revision.Feb 25 2026, 3:38 PM Comment ActionsMakes sense to me too, but the last word is portmgr 's. royger added a comment.Feb 27 2026, 7:43 AM Comment ActionsCan I please get an Ack/Nack from portmgr? This is required for SeaBIOS to be functional again when built on FreeBSD >= 15. Comment Actions I don't think this should break too many things. royger added a comment.Mar 4 2026, 11:50 AM Comment ActionsDoes it need an exp-run? Because I have no idea how to do that. mat added a comment.Mar 4 2026, 1:26 PM Comment ActionsWell, like everything committer related, it's in the committer's guide https://docs.freebsd.org/en/articles/committers-guide/#ports-exp-run, but I don't see what would break with this change. There are not too many ports with USE_BINUTILS though, you could test build them :-) royger added a comment.Mar 4 2026, 2:20 PM Comment Actions
Right, however my FreeBSD development box died yesterday, and I won't get the replacement parts until Monday to bring it back up. I don't mind waiting, but I also would be very surprised for this to break anything. Ports using USE_BINUTILS must work fine with GNU strip I would think :). royger edited the summary of this revision. (Show Details) Comment ActionsDo not take over the STRIP makefile variable, which is already used in bsd.port.mk to signal whether the install program should strip the binaries. Instead define STRIP_CMD and use STRIP just for the environment variables. This revision now requires review to proceed.Mar 10 2026, 11:38 AM royger added a comment.Mar 11 2026, 7:30 AM Comment ActionsI've tested with this fix and I'm able to build all the USE_BINUTILS ports, except for GCC which I haven't tried because my box runs out of memory when building it. royger added a comment.Mon, Mar 16, 9:39 AM Comment ActionsPing? I would like to get this done sooner rather than later so the SeaBIOS port is functional again. Without this change the generated SeaBIOS firmware image is non-functional. dim accepted this revision.Mon, Mar 16, 10:22 AM Comment ActionsLGTM, if there is any ports fallout it is easy to revert. This revision is now accepted and ready to land.Mon, Mar 16, 10:22 AM
Revision Contents
Diff 173498 |
