Page MenuHomeFreeBSD

Drop 'Set to' from the beginning of most src.conf(5) knobs
ClosedPublic

Authored by emaste on Apr 5 2021, 12:59 AM.
Tags
None
Referenced Files
F108497983: D29583.id87173.largetrue.diff
Sat, Jan 25, 3:07 PM
F108494388: D29583.id87060.diff
Sat, Jan 25, 1:51 PM
F108494017: D29583.id87173.diff
Sat, Jan 25, 1:43 PM
F108485058: D29583.diff
Sat, Jan 25, 10:44 AM
Unknown Object (File)
Tue, Jan 21, 3:45 AM
Unknown Object (File)
Sat, Jan 18, 10:22 PM
Unknown Object (File)
Sat, Jan 18, 5:16 PM
Unknown Object (File)
Sat, Jan 18, 5:12 PM

Details

Summary

The description is obviously what effect the knob has when set, so the additional text was unnecessary and made for a somewhat tortured src.conf(5).

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

tools/build/options/WITHOUT_GH_BC
2

I would maybe avoid making other changes for this commit than just the "Set to" part. Losing the Xr sections for bc and dc seems rather odd.

Granted, I would rewrite this description to be something like "Use the traditional FreeBSD versions of bc(1) and dc(1) rather than the <foo> versions." where <foo> is more descriptive than just "enhanced".

emaste added a subscriber: se.

Update WITHOUT_GH_BC based on jhb feedback

tools/build/options/WITHOUT_GH_BC
2

Huh, I think this must have been an accident during a rebase, I did not intend to drop these. When I first did this it was a bit of sed with manual review/touch-ups. In can incorporate feedback before this gets committed and/or we can make further improvements like this one after commit.

I am also not sure what <foo> would be off hand.

6

and of course I have a typo here. corrected locally.

A very reasonable change - I had first committed exactly the text now suggested for WITH_GH_BC and only later adjusted it to better match the other entries.

And thanks to jhb for checking the patches and finding the issues with the suggested WITHOUT_GH_BC change ...
The same issue with the section numbers removed from .Xr lines applies to WITH_GH_BC, btw.

I'm accepting this review without verification of all entries under the assumption that the other changes listed as affecting only 2 lines do not modify more than planned.

tools/build/options/WITH_GH_BC
3

Please add back the section number "1" here and behind .Xr dc ...

This revision is now accepted and ready to land.Apr 8 2021, 8:08 PM

Oh I see what happened - I carried this change locally from before 68c8c1a705251e2b3a8afc4c70c45610217d6e92 and missed the section number when resolving the conflict.

This revision was automatically updated to reflect the committed changes.