tech-pkg archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: RHEL/Centos 7 builtin vs pkgsrc zlib clash for Makefiles that include devel/zlib/

Jason Bacon <> writes:

> On 2020-08-18 19:35, Brook Milligan wrote:
>> Are all the yes/no options lower case as in your patch?  Shouldn't they be case insensitive?  Or are you matching prior art?
> 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>".
> 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.

Given that the flags that are already there behave a certain way, it
seems clear this patch should not change that, and should have the same
yes/YES behavior as before.

(If people want to talk about changing yes/YES behavior in general, I
don't mean to say "that's prohibited".   Just "please keep separable
changes separable".  (But I also see no reason to change.))

Attachment: signature.asc
Description: PGP signature

Home | Main Index | Thread Index | Old Index