Page MenuHomeFreeBSD

Change the meaning of NO_WRKSUBDIR to force a WRKDIR != WRKSRC.
ClosedPublic

Authored by mat on Jun 4 2015, 8:59 PM.
Tags
None
Referenced Files
F102255346: D2735.diff
Sat, Nov 9, 4:05 PM
Unknown Object (File)
Thu, Nov 7, 3:00 AM
Unknown Object (File)
Sat, Oct 26, 2:00 PM
Unknown Object (File)
Sat, Oct 26, 12:24 AM
Unknown Object (File)
Sat, Oct 26, 12:23 AM
Unknown Object (File)
Sat, Oct 26, 12:16 AM
Unknown Object (File)
Sat, Oct 26, 12:16 AM
Unknown Object (File)
Sat, Oct 26, 12:16 AM
Subscribers

Details

Summary

Right now, NO_WRKSUBDIR means that the extraction does not produce a subdirectory, and that everything is living in WRKDIR. It is problematic, because during the build of a port, quite a few files are created in there, and then, a stage directory, where everything is installed, and then a pkg directory where the package is created.

With this, NO_WRKSUBDIR will extract the distfiles directly into WRKSRC instead of WRKDIR.

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 meaning of NO_WRKSUBDIR to force a WRKDIR != WRKSRC..
mat updated this object.
mat edited the test plan for this revision. (Show Details)
bapt added a reviewer: bapt.
bapt added a subscriber: bapt.

accepted only if it passes an exp-run

This revision is now accepted and ready to land.Jun 4 2015, 9:04 PM
mat edited edge metadata.

Update the documentation.

This revision now requires review to proceed.Jun 4 2015, 9:14 PM
mat edited edge metadata.

Include lha and zip in the use of EXTRACT_WRKDIR.

antoine added inline comments.
Mk/bsd.port.mk
529 ↗(On Diff #9735)

there is a strange character in this line

Mk/bsd.port.mk
529 ↗(On Diff #9735)

Mmm, yes, there was a non breaking space, fixed.

  • some non breaking space got included.
mat marked 2 inline comments as done.Oct 28 2015, 7:44 PM
  • Fix linux ports, I think.

Rebase and reduce commits.

This revision was automatically updated to reflect the committed changes.
This revision was automatically updated to reflect the committed changes.