Page MenuHomeFreeBSD

universe: allow moving build logs to UNIVERSE_LOGDIR
AcceptedPublic

Authored by siva on Feb 27 2026, 4:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 14, 1:39 PM
Unknown Object (File)
Thu, May 14, 3:46 AM
Unknown Object (File)
Wed, May 13, 8:40 PM
Unknown Object (File)
Wed, May 13, 8:38 PM
Unknown Object (File)
Wed, May 13, 6:29 PM
Unknown Object (File)
Wed, May 13, 6:29 PM
Unknown Object (File)
Mon, May 11, 7:13 AM
Unknown Object (File)
Mon, May 11, 7:13 AM
Subscribers

Details

Summary

This allows make universe or make tinderbox
to build from a read-only src tree.

Diff Detail

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

Event Timeline

siva requested review of this revision.Feb 27 2026, 4:51 PM
siva created this revision.

I actually find it really convenient that the log files are in .CURDIR.

I think the convenience is nice for sure, but the flexibility of choice is reduced. If anything, maybe I can add a variable similar to FAILFILE that is configurable so that users can choose where it goes?

I do like the idea of being able to build from a read-only src tree.

How about this instead? This keeps the old behaviour by default
for backwards compatibility.

siva retitled this revision from universe: move build logs to OBJROOT to universe: allow moving build logs to TINDERBOX_LOGDIR.Apr 7 2026, 8:58 PM
delphij added a subscriber: delphij.

Looks fine to me.

This revision is now accepted and ready to land.Apr 9 2026, 12:35 AM

My only suggestion would be to be consistent and apply this to make universe as well. That is, I would call this UNIVERSE_LOGDIR and move it out of the DOING_TINDERBOX check. This should also be documented in build(7), but I like the idea overall.

siva retitled this revision from universe: allow moving build logs to TINDERBOX_LOGDIR to universe: allow moving build logs to UNIVERSE_LOGDIR.

Thanks, that makes more sense. Updated the diff now, and also removed an unnecessary check against DOING_TINDERBOX because rm -f succeeds when the file doesn't exist anyway.

This revision now requires review to proceed.Fri, Apr 24, 3:47 PM

Added documentation in build(7).

This revision is now accepted and ready to land.Wed, May 6, 9:19 PM