Page MenuHomeFreeBSD
Differential D56271

packages: Always use the bootstrap flua
ClosedPublic
Actions

Authored by ivy on Mon, Apr 6, 1:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 18, 9:47 PM
Unknown Object (File)
Sat, Apr 18, 11:51 AM
Unknown Object (File)
Sat, Apr 18, 11:49 AM
Unknown Object (File)
Sat, Apr 18, 9:38 AM
Unknown Object (File)
Sat, Apr 18, 3:46 AM
Unknown Object (File)
Fri, Apr 17, 10:52 AM
Unknown Object (File)
Thu, Apr 16, 1:56 PM
Unknown Object (File)
Wed, Apr 15, 7:22 PM
View All 14 Files
Subscribers
emaste
imp
jrtc27

Details

Reviewers
kevans
emaste
Group Reviewers
pkgbase
Commits
rG1ff7f5562d64: packages: Always use the bootstrap flua
rG359d6e31afe3: packages: Always use the bootstrap flua
Summary

To avoid breaking the build due to incompatible changes in flua or lua
libraries on the host, use the bootstrap flua for the package build.

MFC after: 2 weeks
Sponsored by: https://www.patreon.com/bsdivy

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 71987
Build 68870: arc lint + arc unit

Event Timeline

ivy created this revision.Mon, Apr 6, 1:57 PM
Herald added subscribers: jrtc27, emaste, imp. · View Herald TranscriptMon, Apr 6, 1:57 PM
ivy requested review of this revision.Mon, Apr 6, 1:57 PM
Harbormaster completed remote builds in B71987: Diff 174964.Mon, Apr 6, 1:57 PM
ivy added a parent revision: D56270: flua: Always build as a bootstrap tool.Mon, Apr 6, 1:57 PM
kevans accepted this revision.Mon, Apr 6, 1:59 PM
This revision is now accepted and ready to land.Mon, Apr 6, 1:59 PM
emaste accepted this revision.Mon, Apr 6, 2:04 PM

I think this is sensible. Could this also be achieved via USE_STRICT_TMPPATH? I'd like to see that (eventually) become the default, and we could remove the PATH= setting then.

ivy added a comment.Mon, Apr 6, 2:14 PM
In D56271#1287496, @emaste wrote:

I think this is sensible. Could this also be achieved via USE_STRICT_TMPPATH? I'd like to see that (eventually) become the default, and we could remove the PATH= setting then.

discussed on irc, but just to leave the answer here -- yes, we could eventually do that, although it may become redundant once i adapt this patch to D56087 (which is going to be delayed due to this issue).

Closed by commit rG359d6e31afe3: packages: Always use the bootstrap flua (authored by ivy). · Explain WhyTue, Apr 7, 11:04 AM
This revision was automatically updated to reflect the committed changes.
ivy added a commit: rG359d6e31afe3: packages: Always use the bootstrap flua.
ivy added a commit: rG1ff7f5562d64: packages: Always use the bootstrap flua.Sun, Apr 12, 11:15 PM

Revision Contents
Changeset List

PathSize
21 lines
release/
packages/
2 lines
2 lines

Diff 174964

View Options

Makefile.inc1

Loading...
View Options

release/packages/create-sets.sh

Loading...
View Options

release/packages/generate-ucl.sh

Loading...