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)
Fri, Oct 2, 4:29 PM
Unknown Object (File)
Mon, Sep 21, 10:29 AM
Unknown Object (File)
Sun, Sep 20, 2:14 AM
Unknown Object (File)
Fri, Sep 18, 11:03 PM
Unknown Object (File)
Fri, Sep 18, 10:13 PM
Unknown Object (File)
Thu, Sep 17, 8:41 AM
Unknown Object (File)
Wed, Sep 16, 12:55 PM
Unknown Object (File)
Fri, Sep 11, 10:34 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.