Page MenuHomeFreeBSD

bsd.mkopt.mk: Undefine __REQUIRED_OPTIONS
AbandonedPublic

Authored by des on Aug 8 2025, 1:36 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Sep 6, 11:26 PM
Unknown Object (File)
Sun, Sep 6, 3:32 AM
Unknown Object (File)
Thu, Sep 3, 10:35 PM
Unknown Object (File)
Wed, Aug 26, 12:56 PM
Unknown Object (File)
Thu, Aug 13, 8:04 PM
Unknown Object (File)
Thu, Aug 13, 8:03 PM
Unknown Object (File)
Jul 25 2026, 6:10 PM
Unknown Object (File)
Jul 23 2026, 4:57 PM

Details

Reviewers
None
Group Reviewers
srcmgr
Summary

The other three (__DEFAULT_{YES,NO,DEPENDENT}_OPTIONS) were already
being undefined, but this one got missed.

Diff Detail

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

Event Timeline

des requested review of this revision.Aug 8 2025, 1:36 PM

tools/build/options/makeman tries to extract the value of __REQUIRED_OPTIONS using make -V. Are you sure this doesn't break that?

Ah, it does, but we should really find a different way of getting that information (or even hardcode it in the script). Not undefining __REQUIRED_OPTIONS here means we can only use it in one place; since we already use it in src.opts.mk, we can't ever use it in bsd.opts.mk.

Folded into D51827 which also includes a fix for makeman.