Page MenuHomeFreeBSD

make only vital packages vital, not their sub-packages
ClosedPublic

Authored by freebsd_igalic.co on Mar 11 2021, 8:33 PM.
Referenced Files
F82028721: D29224.id85989.diff
Wed, Apr 24, 6:13 PM
F82007127: D29224.id85598.diff
Wed, Apr 24, 11:00 AM
Unknown Object (File)
Mon, Apr 22, 1:26 AM
Unknown Object (File)
Mon, Apr 22, 12:55 AM
Unknown Object (File)
Sat, Apr 20, 2:21 AM
Unknown Object (File)
Sun, Apr 7, 4:12 PM
Unknown Object (File)
Mar 22 2024, 7:40 AM
Unknown Object (File)
Mar 22 2024, 7:40 AM
Tokens
"Love" token, awarded by ehem_freebsd_m5p.com.

Details

Summary

make "vital" a replaceable, which defaults to "false" and only set it for
the main clib, utilities and runtime packages, not their sub-packages

PR: 254174

Test Plan

i have not tested this yet.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 37930
Build 34819: arc lint + arc unit

Event Timeline

I don't think that the jail package should be vital.
Otherwise that looks good.

release/packages/generate-ucl.sh
41

clibs should be vital too.

release/packages/generate-ucl.sh
56

well, this is wrong

59

and why are jails vital?

addressed in latest update

freebsd_igalic.co retitled this revision from make jail, runtimes & utilities sub-packages non-vital to make only vital packages vital, not their sub-packages.Mar 12 2021, 11:59 AM
freebsd_igalic.co edited the summary of this revision. (Show Details)
freebsd_igalic.co edited the test plan for this revision. (Show Details)

Looks ok,
Can you rebase your patch against latest main ?
Thanks.

This revision was not accepted when it landed; it landed in state Needs Review.Mar 28 2021, 10:19 AM
This revision was automatically updated to reflect the committed changes.

I'm unsure which commit did it, but at a point near this commit the time taken by release builds drastically decreased. Before this close to 20 hours for an aarch64 cross-build, just over 6 after. Even though I can leave my build machine running, the decrease in latency is very nice. Thank you.