On 2020-08-18 19:35, Brook Milligan wrote:
Good question. Manually set mk variables can certainly take case-insensitive values, but I think the command-line flags should agree with the usage message: "--prefer-pkgsrc <list|yes|no>".Are all the yes/no options lower case as in your patch? Shouldn't they be case insensitive? Or are you matching prior art?
If there's a consensus that the boostrap script should accept things like "--prefer-pkgsrc YES" or "--prefer-pkgsrc Yes", I would be happy to change it and update the usage message to match.
I also occurs to me now that we can never have a package named "yes" or "no" without breaking these flags.