Paths

Table of Contentst

Differential D56271

packages: Always use the bootstrap flua
ClosedPublic
Actions

Authored by ivy on Mon, Apr 6, 1:57 PM.

Tags

None

Referenced Files

	F152839472: D56271.id.diff
	Fri, Apr 17, 10:52 AM

	F152696153: D56271.id174964.diff
	Thu, Apr 16, 1:56 PM

	Unknown Object (File)
	Wed, Apr 15, 7:22 PM

	Unknown Object (File)
	Wed, Apr 15, 10:48 AM

	Unknown Object (File)
	Wed, Apr 15, 12:07 AM

	Unknown Object (File)
	Sun, Apr 12, 6:06 AM

	Unknown Object (File)
	Fri, Apr 10, 11:34 PM

	Unknown Object (File)
	Thu, Apr 9, 6:59 PM

View All 9 Files

Subscribers

Details

Reviewers

kevans
emaste

Group Reviewers

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 Not Applicable

Unit

Tests Not Applicable

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

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.

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

Path

Size

21 lines

release/

packages/

2 lines

generate-ucl.sh

2 lines

Diff 175023

Makefile.inc1

Loading...

release/packages/create-sets.sh

Loading...

release/packages/generate-ucl.sh

Loading...