Page MenuHomeFreeBSD
Differential D39742

bsdinstall avoid subdir depending on parent
ClosedPublic
Actions

Authored by sjg on Apr 21 2023, 3:57 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Oct 8, 1:36 PM
Unknown Object (File)
Sun, Sep 20, 12:41 PM
Unknown Object (File)
Fri, Sep 18, 2:24 PM
Unknown Object (File)
Sep 9 2026, 11:29 PM
Unknown Object (File)
Sep 6 2026, 10:25 AM
Unknown Object (File)
Sep 5 2026, 12:22 PM
Unknown Object (File)
Sep 5 2026, 9:02 AM
Unknown Object (File)
Sep 5 2026, 5:51 AM
View All Files
Subscribers
None

Details

Reviewers
stevek
imp
bdrewery
obrien
Commits
rG0c3627f44d49: bsdinstall avoid subdir depending on parent
Summary

When not doing tree walks, it is bad for sub-dirs to depend on
parents. Move the generation of opt_osname.h to distextract
and have others that need that depend on it.

In usr.sbin/bsdinstall use SUBDIR_DEPEND_ so tree walking still works.

Diff Detail

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

Event Timeline

sjg requested review of this revision.Apr 21 2023, 3:57 AM
sjg created this revision.
Harbormaster completed remote builds in B51087: Diff 120786.Apr 21 2023, 3:57 AM
obrien added a reviewer: obrien.Apr 21 2023, 4:01 AM
obrien added a subscriber: obrien.
obrien removed a subscriber: obrien.

Looks good.

obrien accepted this revision.Apr 21 2023, 4:03 AM
This revision is now accepted and ready to land.Apr 21 2023, 4:03 AM
Closed by commit rG0c3627f44d49: bsdinstall avoid subdir depending on parent (authored by sjg). · Explain WhyApr 21 2023, 5:02 AM
This revision was automatically updated to reflect the committed changes.
sjg added a commit: rG0c3627f44d49: bsdinstall avoid subdir depending on parent.

Revision Contents
Changeset List

PathSize
usr.sbin/
bsdinstall/
9 lines
distextract/
11 lines
distfetch/
2 lines
partedit/
2 lines

Diff 120786

View Options

usr.sbin/bsdinstall/Makefile

Loading...
View Options

usr.sbin/bsdinstall/distextract/Makefile

Loading...
View Options

usr.sbin/bsdinstall/distfetch/Makefile

Loading...
View Options

usr.sbin/bsdinstall/partedit/Makefile

Loading...