Re: [evolution-patches] Patch: kerberos library location patch



Looks good now. :)

-- dobey

On Fri, 2004-08-13 at 16:38 +0200, Frederic Crozat wrote:
> Le jeu 12/08/2004 �7:28, Rodney Dawes a �it :
> > On Wed, 2004-08-11 at 16:43 +0200, Frederic Crozat wrote:
> > > As part of our AMD64 port, here is a patch I'd like to merge in HEAD
> > > branch (I have a similar one for 1.4 branch, but I don't think it is
> > > worth merging it).
> > > 
> > > It is needed when dynamic library are stored in $prefix/lib64, as they
> > > are on AMD64 platform.
> > > 
> > > Permission to commit ?
> > 
> > 
> > > +2004-03-26  Frederic Crozat  <fcrozat mandrakesoft com>
> > 
> > Hrmm. You should cvs up first. ;)
> > 
> > 
> > > +AC_ARG_WITH(krb5, [  --with-krb5=PREFIX      Location of Kerberos 5
> > > install dir], with_krb5="$withval", with_krb5="no")
> > > +AC_ARG_WITH(krb5_libs, [  --with-krb5-libs=LIBDIR Location of
> > > Kerberos 5 libs], with_krb5_libs="$withval",
> > > with_krb5_libs="$with_krb5/lib")
> > > +AC_ARG_WITH(krb4, [  --with-krb4=PREFIX      Location of Kerberos 4
> > > install dir], with_krb4="$withval", with_krb4="no")
> > > +AC_ARG_WITH(krb4_libs, [  --with-krb4-libs=LIBDIR Location of
> > > Kerberos 4 libs], with_krb4_libs="$withval",
> > > with_krb4_libs="$with_krb4/lib")
> > 
> > Should use "installation" instead of "install dir" in the summaries,
> > as well as "libraries" instead of "libs". We should also have similar
> > options for --with-FOO-includes, to be complete. Also, I believe that
> > the first option to AC_ARG_WITH here, for the -libs options, needs to
> > be FOO-libs instead of FOO_libs. Stylistically, I'd also like to see
> > the help summary use =DIR instead of =PREFIX or =LIBDIR.
> > 
> > The rest of the actual functionality looks ok though.
> 
> New version attached..
> 




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