Page MenuHomeFreeBSD

Fix double settings of vars due to being done before bsd.port.post.mk.
ClosedPublic

Authored by bdrewery on Mar 22 2015, 8:17 PM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 19 2023, 11:51 PM
Unknown Object (File)
Nov 6 2023, 10:17 PM
Unknown Object (File)
Oct 5 2023, 9:16 PM
Unknown Object (File)
Sep 14 2023, 1:46 AM
Unknown Object (File)
Sep 13 2023, 5:53 PM
Unknown Object (File)
May 25 2023, 2:25 PM
Unknown Object (File)
Dec 24 2022, 9:01 AM
Unknown Object (File)
Mar 4 2017, 7:40 PM
Subscribers

Details

Reviewers
bapt
Group Reviewers
portmgr
Summary

Fix duplicate values in ENV vars and X_BUILD_FOR truncating BUILD_DEPENDS.

PKG_NOTES/PKG_ENV/CONFIGURE_ENV/MAKE_ENV/CO_ENV/QA_ENV were using += before the
double include check so any port using bsd.port.options.mk or bsd.port.pre.mk
would have duplicated items in these vars.

All ports would have their BUILD_DEPENDS truncated with X_BUILD_FOR.

Test Plan

Local make -V testing with bsd.port.pre.mk / bsd.port.options.mk and setting X_BUILD_FOR. Local poudriere testing.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

bdrewery retitled this revision from to Fix double settings of vars due to being done before bsd.post.port.mk..
bdrewery updated this object.
bdrewery edited the test plan for this revision. (Show Details)
bdrewery added a reviewer: portmgr.
bdrewery retitled this revision from Fix double settings of vars due to being done before bsd.post.port.mk. to Fix double settings of vars due to being done before bsd.port.post.mk..Mar 22 2015, 8:18 PM
bdrewery edited the test plan for this revision. (Show Details)
bdrewery edited edge metadata.
  • Move X_BUILD_FOR CC/etc handling back to the top.
  • Move bsd.port.mk double inclusion warning back to right after the .ifdef section.
bapt added a reviewer: bapt.
bapt added a subscriber: bapt.

That would need an exp-run otherwise looks good

This revision is now accepted and ready to land.Mar 22 2015, 8:27 PM