VOOZH about

URL: https://reviews.freebsd.org/D55509

⇱ ⚙ D55509 bsd.port.mk: also export GNU strip when USE_BINUTILS is set


bsd.port.mk: also export GNU strip when USE_BINUTILS is set
AcceptedPublic

Authored by royger on Feb 25 2026, 2:19 PM.
Tags
None
Referenced Files
F151801814: D55509.diff
Fri, Apr 10, 6:44 PM
Unknown Object (File)
Wed, Apr 8, 10:20 PM
Unknown Object (File)
Sun, Apr 5, 1:11 PM
Unknown Object (File)
Sat, Apr 4, 8:52 AM
Unknown Object (File)
Sat, Apr 4, 8:52 AM
Unknown Object (File)
Sat, Apr 4, 8:52 AM
Unknown Object (File)
Sat, Apr 4, 7:53 AM
Unknown Object (File)
Sat, Apr 4, 7:53 AM
Subscribers
None

Details

Reviewers
bapt
dim
mat
delphij
arrowd
cy
Group Reviewers
portmgr
Summary

GNU Binutils packages strip, and should be used in preference to the base
LLVM strip when USE_BINUTILS = yes.

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

Repository
R11 FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

royger created this revision.
Comment Actions

Sounds 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
Comment Actions

Picked 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.

Comment Actions

Makes sense to me too, but the last word is portmgr 's.

Comment Actions

Can 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.

Comment Actions

Does it need an exp-run? Because I have no idea how to do that.

Comment Actions

Well, 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 :-)

Comment Actions
In D55509#1273416, @mat wrote:

Well, 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 :-)

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 Actions

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.

This revision now requires review to proceed.Mar 10 2026, 11:38 AM
Comment Actions

I'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.

Comment Actions

Ping? 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.

Comment Actions

LGTM, 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

PathSize
Mk/
8 lines

Diff 173498

Mk/bsd.port.mk

Loading...