Page MenuHomeFreeBSD

Centralize definition of ZFSTOP
Needs ReviewPublic

Authored by jhb on Mon, Nov 17, 5:31 PM.
Tags
None
Referenced Files
F136750058: D53790.id.diff
Wed, Nov 19, 6:26 AM
F136750053: D53790.id166586.diff
Wed, Nov 19, 6:26 AM
F136750037: D53790.diff
Wed, Nov 19, 6:26 AM
F136663023: D53790.id166586.diff
Tue, Nov 18, 7:24 PM
F136662061: D53790.diff
Tue, Nov 18, 7:18 PM
F136604485: D53790.id166586.diff
Tue, Nov 18, 1:29 PM
Unknown Object (File)
Tue, Nov 18, 7:15 AM
Unknown Object (File)
Tue, Nov 18, 5:36 AM

Details

Reviewers
mm
sjg
Group Reviewers
cheri
Summary

Define this helper variable in one place and make it available while
building the entire base system.

Obtained from: CheriBSD (mostly)

Diff Detail

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

Event Timeline

jhb requested review of this revision.Mon, Nov 17, 5:31 PM

In CheriBSD we have a second import of ZFS (with various CHERI patches applied) which is why we benefit from having a single definition of this variable, but I think the cleanup of using ZFSTOP more widely is nice regardless as it makes several long lines a bit shorter.

share/mk/src.sys.mk
51

@sjg Is this the right place to put a definition for a variable that is shared across the entire src tree?

Very happy to have the duplication removed

share/mk/src.sys.mk
51

It is certainly not unreasonable. There are potentially many makefiles included before this, if you want to allow any of those to provide an alternative, you might want to use ?= here.

Eg. in our trees at work (with site.*.mk) there are 29 makefiles read before src.sys.mk, even in upstream main, there are 17 makefiles read first.