MFC after: 3 days
Reported by: adamw
Details
Details
Diff Detail
Diff Detail
- Repository
- rG FreeBSD src repository
- Lint
Lint Skipped - Unit
Tests Skipped - Build Status
Buildable 71231 Build 68114: arc lint + arc unit
Event Timeline
Comment Actions
I like this, but version … version reads a little oddly.
What about using “variant” for one of them? (Gotta credit my wife for that idea; I drew a blank lol)
Comment Actions
I spent a looooong time with ChatGPT working on phrasing. Why is this one so tough?!
What I came up with was: "Override the defaults for ports with selectable variants."?
Depending on length target, we could really clear it up for users by adding "such as database and compiler versions."
I dunno, folks. Maybe we should just say "DEFAULT_VERSIONS: Sets default versions" 🙄
Comment Actions
use database or compiler versions, i like that.
i like the text we've come up with so far otherwise.
Comment Actions
"Variant ... variant" didn't feel good, but with the rest of the sentence, it now reads well. I like this change and I say go for it!
Comment Actions
Oops, the multiple variants was a typo. I actually want to use version
for the second one. Thanks for your attention to detail here!