etcupdate: Warn if the tree hasn't been built
ClosedPublic
Actions

Authored by des on Jun 8 2026, 4:38 PM.

Details

Reviewers

jhb
jrtc27

Commits

rG53a3d4760eb9: etcupdate: Warn if the tree hasn't been built

Summary

Warn the user before trying make installetc if etcupdate was invoked
without -b (or with -B) and it appears that make buildetc hasn't
already been run (which usually happens as part of make buildworld).

MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

des created this revision.Jun 8 2026, 4:38 PM

Herald added a subscriber: imp. · View Herald TranscriptJun 8 2026, 4:38 PM

des requested review of this revision.Jun 8 2026, 4:38 PM

Harbormaster completed remote builds in B73748: Diff 179395.Jun 8 2026, 4:38 PM

jhb added inline comments.Jun 18 2026, 1:08 PM

usr.sbin/etcupdate/etcupdate.sh
228	Should this bard fail instead of just being a warning?
1913	To avoid ambiguity with an output file that `etcupdate build` places into a tree.

des marked an inline comment as done.Jun 18 2026, 1:22 PM

des added inline comments.

usr.sbin/etcupdate/etcupdate.sh
228	If one day the build stops producing the file we chose as a sentinel and nobody updates `etcupdate`, failing here would prevent `etcupdate` from running for no good reason. Not failing here, however, simply leads to failing in `make installetc` a little while later. Since the output goes to the log, the warning remains visible on the console when that happens.