[Buildroot] [PATCH v2, 1/2] package/libkrb5: add openssl optional dependency
Peter Korsgaard
peter at korsgaard.com
Mon Jun 21 19:08:36 UTC 2021
>>>>> "Yann" == Yann E MORIN <yann.morin.1998 at free.fr> writes:
> Fabrice, All,
> On 2021-06-13 18:41 +0200, Fabrice Fontaine spake thusly:
>> Le dim. 13 juin 2021 à 18:08, Yann E. MORIN <yann.morin.1998 at free.fr> a écrit :
>> > On 2021-06-13 11:10 +0200, Fabrice Fontaine spake thusly:
>> > > openssl is an optional dependency which is enabled by default since at
>> > > least 2007 and
>> > > https://github.com/krb5/krb5/commit/4c17f25c0f106ec8e3aad2cee4e7a24f66b5a65d
>> > >
>> > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice at gmail.com>
>> > I squashed the two commits together, because selecting the openssl
>> > options is really just part of fixing the openssl dependency of libkrb5.
>> Perhaps, it would have been better to keep them separate as Peter will
>> probably only want to backport the first patch to 2020.xx and 2021.xx
>> branches?
> That's true.
> However, semantically, I still believe the two patches should really
> have been one, because if we don't select the options when we explicitly
> add the openssl depenency, then we are still introducing a failure.
> And I don't think that it is unreasonable to expect that some commits
> have to be adapted when backported.
Indeed. Committed to 2021.02.x and 2021.05.x, after dropping the openssl
options - thanks.
--
Bye, Peter Korsgaard
More information about the buildroot
mailing list