Page MenuHomeFreeBSD
Differential D17221

Hoist LDFLAGS_${ARCH} into bsd.port.mk
ClosedPublic
Actions

Authored by emaste on Sep 18 2018, 2:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 19, 8:08 AM
Unknown Object (File)
Tue, Nov 19, 8:00 AM
Unknown Object (File)
Tue, Nov 19, 7:41 AM
Unknown Object (File)
Dec 22 2023, 10:11 PM
Unknown Object (File)
Dec 13 2023, 8:30 AM
Unknown Object (File)
Nov 24 2023, 2:55 AM
Unknown Object (File)
Sep 2 2023, 9:10 AM
Unknown Object (File)
Jun 19 2023, 8:33 PM
View All 13 Files
Subscribers
mat
mmokhi

Details

Reviewers
bapt
Group Reviewers
portmgr

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

emaste created this revision.Sep 18 2018, 2:41 PM
bapt accepted this revision.Sep 18 2018, 2:42 PM
This revision is now accepted and ready to land.Sep 18 2018, 2:42 PM
emaste added a comment.Sep 18 2018, 2:49 PM

rP480026

emaste closed this revision.Sep 18 2018, 2:49 PM
mmokhi added a subscriber: mmokhi.Sep 18 2018, 3:44 PM
mmokhi added a comment.Sep 18 2018, 4:17 PM

while we are here I'd like to mention this relevant review which maybe some people have opinions on as well 😅

emaste added a comment.Sep 18 2018, 4:20 PM
In D17221#367105, @mmokhi wrote:

while we are here I'd like to mention this relevant review which maybe some people have opinions on as well 😅

Indeed, that approach would be convenient but some folks didn't like it :)

That said, I now think it doesn't matter much: LDFLAGS_i386=-Wl,-znotext perhaps has less semantic meaning, but is just as much of a nop as D16817 when ld.bfd is in use.

mat added a subscriber: mat.Sep 18 2018, 5:03 PM
mat added inline comments.
Mk/bsd.port.mk
2065

technically, the .if is useless, and if it was supposed to be meaningful, it should probably be .if !empty(LDFLAGS_${ARCH})

emaste marked an inline comment as done.Sep 18 2018, 5:10 PM
emaste added inline comments.
Mk/bsd.port.mk
2065

I was just following the approach for ${lang}FLAGS_${ARCH} above. Shall I take the .if out?

emaste marked an inline comment as done.Sep 18 2018, 5:11 PM
emaste added inline comments.
Mk/bsd.port.mk
2065

(Please explicit Approved by: portmgr if so)

mat added inline comments.Sep 18 2018, 5:35 PM
Mk/bsd.port.mk
2065

Yes, please, and the .if on line 2060, approved by portmgr.

Revision Contents
Changeset List

PathSize
Mk/
5 lines
archivers/
p7zip/
1 line
multimedia/
libx264/
2 lines
xmms/
2 lines

Diff 48166

View Options

Mk/bsd.port.mk

Loading...
View Options

archivers/p7zip/Makefile

Loading...
View Options

multimedia/libx264/Makefile

Loading...
View Options

multimedia/xmms/Makefile

Loading...