[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