Page MenuHomeFreeBSD

Change the way opt_USE is worked on, so that it is not evaluated too early.
ClosedPublic

Authored by mat on Oct 7 2016, 7:03 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 19 2024, 5:11 AM
Unknown Object (File)
Mar 19 2024, 2:33 AM
Unknown Object (File)
Dec 28 2023, 5:14 AM
Unknown Object (File)
Dec 28 2023, 5:14 AM
Unknown Object (File)
Dec 28 2023, 5:14 AM
Unknown Object (File)
Dec 26 2023, 5:10 PM
Unknown Object (File)
Dec 25 2023, 10:51 AM
Unknown Object (File)
Dec 20 2023, 12:11 AM
Subscribers

Details

Reviewers
brnrd
Group Reviewers
portmgr
Commits
rP423859: Change the way opt_USE is parsed.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

mat retitled this revision from to Change the way opt_USE is worked on, so that it is not evaluated too early..
mat updated this object.
mat edited the test plan for this revision. (Show Details)
Mk/bsd.options.mk
500 โ†—(On Diff #21131)

Would a check here and on line 550 be useful to determine if USE_${_u:tu} already contains _u?

For example,

OPT1_USE= ldconfig=${PREFIX}/lib/blah/lib
OPT2_USE= ldconfig=${PREFIX}/lib/blah/lib

results in USE_LDCONFIG containing the same path twice.

brnrd edited edge metadata.

Tested this and it results in the same behavior as an empty USE_LDCONFIG= .

This revision is now accepted and ready to land.Oct 7 2016, 7:21 PM
Mk/bsd.options.mk
500 โ†—(On Diff #21131)

Well, in this case, you are asking for it to be put twice.

What you want to do is:

.if ${PORT_OPTIONS:MOPT1} || ${PORT_OPTIONS:MOPT2}
USE_LDCONFIG+= ${PREFIX}/lib/blah/lib
.endif

This revision was automatically updated to reflect the committed changes.