Page MenuHomeFreeBSD

release: switch the WITH_* knobs to just gating `release` dependencies
Needs ReviewPublic

Authored by kevans on Sat, Sep 26, 3:57 AM.
Tags
None
Referenced Files
F176136318: D60036.diff
Fri, Oct 16, 9:38 AM
F176092899: D60036.diff
Fri, Oct 16, 1:21 AM
Unknown Object (File)
Tue, Oct 13, 9:54 PM
Unknown Object (File)
Tue, Oct 13, 9:42 PM
Unknown Object (File)
Mon, Oct 12, 8:39 AM
Unknown Object (File)
Mon, Oct 12, 12:15 AM
Unknown Object (File)
Sat, Oct 10, 10:40 AM
Unknown Object (File)
Fri, Oct 9, 11:18 PM

Details

Reviewers
None
Group Reviewers
Klara
releng
Summary
The knobs are useful to have to control what `make release` does by
default, but it's also quite possible to just drive individual targets
if you don't plan to generate the entire set of release images.  Always
including the logic and only attaching the *-release dependencies onto
the `release` target if the relevant WITH_* knobs are set makes a lot of
sense.

Sponsored by:	Klara, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

When you get to cloudware, my sense is that it's more likely someone will run e.g. make ec2ami-desktop-ufs rather than make cloudware-release.

Perhaps a better solution would be to have the relevant Makefile logic included unconditionally and have the WITH_* flags only control which targets are dependencies of release ?

When you get to cloudware, my sense is that it's more likely someone will run e.g. make ec2ami-desktop-ufs rather than make cloudware-release.

Perhaps a better solution would be to have the relevant Makefile logic included unconditionally and have the WITH_* flags only control which targets are dependencies of release ?

Sure thing, I like that approach a lot more but I wasn't sure if I'd be able to get buy-in (because I might be overlooking how these were expecting to be used). I will do that instead.

Can you also update release(7) with the new knobs?

Can you also update release(7) with the new knobs?

This isn't introducing any new knobs, just imbuing them wth a slightly different meaning that isn't actually incompatible with the currernt verbiage.

kevans retitled this revision from release: builld cloud/vm/oci images if explicitly requested to release: switch the WITH_* knobs to just gating `release` dependencies.
kevans edited the summary of this revision. (Show Details)

Switch gears and just control the release and install dependencies. Convert .WAIT into .ORDER and dcument the pain point that requires it, but call it out of scope for the moment as long as we don't break re@ builds.