buildworld: drop config(8) dependencies from bootstrap
ClosedPublic
Actions

Authored by kevans on Jan 30 2023, 3:54 PM.

Details

Reviewers

imp
jrtc27

Commits

rGb4f7a31fd16b: buildworld: drop config(8) dependencies from bootstrap

Summary

We may still need them for other parts, so just remove the dependency
relationship for now and simplify config's place in bootstrap-tools.

Sponsored by: Klara, Inc.
Sponsored by: NetApp, Inc.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kevans created this revision.Jan 30 2023, 3:54 PM

Herald added a subscriber: emaste. · View Herald TranscriptJan 30 2023, 3:54 PM

kevans requested review of this revision.Jan 30 2023, 3:54 PM

Harbormaster completed remote builds in B49325: Diff 116047.Jan 30 2023, 3:54 PM

kevans added a parent revision: D38276: config: drop dependency on libsbuf.Jan 30 2023, 3:54 PM

this looks fine. I have no caveats on my review here. config remains a bootstrap tool, and this just adjusts its depends.
Only issue would be if that causes something *ELSE* to fail that should have had these depends but we missed that because config did.
push to github to get the cross builds and that should mitigate the risk from this potential issue though.

This revision is now accepted and ready to land.Jan 30 2023, 4:52 PM

Herald added a subscriber: dsl. · View Herald TranscriptJan 30 2023, 4:52 PM

Closed by commit rGb4f7a31fd16b: buildworld: drop config(8) dependencies from bootstrap (authored by kevans). · Explain WhyFeb 9 2023, 5:28 AM

This revision was automatically updated to reflect the committed changes.

kevans added a commit: rGb4f7a31fd16b: buildworld: drop config(8) dependencies from bootstrap.