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
Unknown Object (File)
Feb 18 2024, 10:53 AM
Unknown Object (File)
Jan 16 2024, 5:32 PM
Unknown Object (File)
Jan 12 2024, 2:43 AM
Unknown Object (File)
Dec 23 2023, 4:54 AM
Unknown Object (File)
Dec 2 2023, 10:09 PM
Unknown Object (File)
Nov 29 2023, 11:12 AM
Unknown Object (File)
Nov 29 2023, 5:38 AM
Unknown Object (File)
Nov 10 2023, 7:14 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

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tools/build/options/WITHOUT_GH_BC
3

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
3

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
4

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.