On Sun, Aug 22, 2010 at 15:36:50 -0400, Roberto C. Sánchez wrote:
> On Sun, Aug 22, 2010 at 02:28:37PM -0400, Roberto C. Sánchez wrote:
> >
> > Assuming that my explanations are satisfactory, did you want to see a
> > new diff for placing the configure options into a variable?
> >
> I went ahead and finished making the changes just now. The updated diff
> is attached.
>
I should have spotted this last time, but I'm afraid this:
+ (mkdir $(TMPBUILD_MIT); cd $(TMPBUILD_MIT); \
+ LDFLAGS="$(LDFLAGS) -L/usr/lib/mit-krb5 -Wl,-z,defs"
CFLAGS="$(CFLAGS)" CPPFLAGS="$(CPPFLAGS) -I/usr/include/mit-krb5"
../configure \
+ $(CONFIGURE_COMMON_OPTIONS) \
+ --with-gss_impl=mit; \
+ cd ..)
is going to succeed even if configure doesn't. Should probably use
something like
mkdir $(TMPBUILD_MIT)
cd $(TMPBUILD_MIT) && \
LDFLAGS="$(LDFLAGS) -L/usr/lib/mit-krb5 -Wl,-z,defs" \
CFLAGS="$(CFLAGS)" CPPFLAGS="$(CPPFLAGS) -I/usr/include/mit-krb5" \
../configure $(CONFIGURE_COMMON_OPTIONS) --with-gss_impl=mit
i.e. without the unnecessary subshell and '; cd ..' after configure.
Probably same with the $(MAKE) invocation.
Cheers,
Julien
Attachment:
signature.asc
Description: Digital signature