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

Re: Request for pre-approval of cyrus-sasl2 upload



Julien Cristau <jcristau@debian.org> writes:
> On Sun, Aug 22, 2010 at 00:21:39 -0400, Roberto C. Sánchez wrote:

>>  libsasl2-modules-gssapi-heimdal.install           |    1 
>>  libsasl2-modules-gssapi-heimdal.lintian-overrides |    2 
>>  patches/0024_allow_detection_of_heimdal.dpatch    |   22 ++++

> That patch looks like a kludge around a broken configure check for
> GSS_C_NT_HOSTBASED_SERVICE.  Can you explain it a bit more?

That's because it's a kludge around a broken configure check for
GSS_C_NT_HOSTBASED_SERVICE.  :)  Upstream is currently using
AC_EGREP_HEADER, but in current versions of Heimdal this is a define, so
egreping the results of the C preprocessor doesn't detect it.

This is a minimal fix suitable for a release freeze.  A better fix would
be for upstream to switch to AC_CHECK_DECL, but that's a larger change and
hence has more risk.  Ideally, one would be able to delete a fair chunk of
the upstream Autoconf macros in this area by using AC_CHECK_DECL
appropriately.

-- 
Russ Allbery (rra@debian.org)               <http://www.eyrie.org/~eagle/>


Reply to: