Ethereal-dev: Re: [Ethereal-dev] Re: [PATCH] acinclude.m4, configure.in

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

From: Joerg Mayer <jmayer@xxxxxxxxx>
Date: Fri, 14 May 2004 12:41:23 +0200
On Fri, May 14, 2004 at 12:00:57PM +0200, Thomas Anders wrote:
> You obviously disagreed with yourself later on ;) and replaced the comment
> I introduced in acinclude.m4 with a suitable fix (which made it into 
> 0.10.4):

Yep. At first glance I misunderstood what the patch was about.

> Thanks for that. I needed another small patch (attached) to make
> "--with-krb5=/path/to/dir" work for me even without krb5-config in PATH, 
> though.

Applied.

> As for further improvements: in case of "--with-krb5=/path/to/dir" we should
> look for krb5-config in /path/to/dir/bin (in addition to PATH) and use it.

Can you please let me konw whether the attached patch is OK?

 Thanks
        Joerg


-- 
Joerg Mayer                                           <jmayer@xxxxxxxxx>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.
Index: acinclude.m4
===================================================================
RCS file: /usr/local/cvsroot/ethereal/acinclude.m4,v
retrieving revision 1.75
diff -p -u -r1.75 acinclude.m4
--- acinclude.m4	14 May 2004 10:33:00 -0000	1.75
+++ acinclude.m4	14 May 2004 10:40:37 -0000
@@ -874,6 +874,8 @@ AC_DEFUN([AC_ETHEREAL_KRB5_CHECK],
 	  # as the compiler and/or linker will search that other
 	  # directory before it searches the specified directory.
 	  #
+	  PATH=$PATH:$krb5_dir/bin
+	  export PATH
 	  ethereal_save_CFLAGS="$CFLAGS"
 	  CFLAGS="$CFLAGS -I$krb5_dir/include"
 	  ethereal_save_CPPFLAGS="$CPPFLAGS"