On Thu, 2017-06-29 at 11:08 +0200, Waldemar Brodkorb wrote: > Hi, > Ian Kent wrote, > > > On Wed, 2017-06-28 at 12:29 +0200, Waldemar Brodkorb wrote: > > > When trying to cross-compile autofs for example with a buildsystem > > > like buildroot, the compilation fails in case the user wants to use > > > libtirpc library as RPC implementation. A hard coded include path > > > in aclocal.m4 is used. Other opensource software like rpcbind or > > > nfs-utils are suing autotools pkgconfig infrastructure to find > > > the libtirpc headers and to pass the correct linker flags. > > > > > > Convert configure.in to use PKG_CHECK_MODULES and remove the > > > hand written autoconf macros from aclocal.m4. > > > > > > To autoreconf the package you need pkg-config or pkgconf installed, > > > which provides the needed autoconf macros in pkg.m4. For an > > > non-automake project a full path to pkg.m4 is required. > > > > > > This fixes cross-compilation of autofs and allows to use > > > alternative C libraries as uClibc-ng without internal RPC > > > support to be used in cross-compiling environments. > > > > > > Signed-off-by: Waldemar Brodkorb <wbx@xxxxxxxxxxx> > > > --- > > > v1 -> v2: > > > - add a better commit decription > > > - add back missing TIRPC_WORKAROUND/WITH_LIBTIRPC defines > > > - add back AC_CHECK_FUNCS([getrpcbyname getservbyname]) > > > > > > For autoheader/autoconf to generate configure/config.h you > > > need to have pkg-config installed on your development machine. > > > When the user does not have installed pkg-config later, he > > > can use autofs without libtirpc as before. > > > Configure will fail when the user use --with-libtirpc and > > > pkg-config is not installed. > > > > Thanks for this, I'll have a look through it. > > > > > +# for pkg-config macros > > > +m4_include([/usr/share/aclocal/pkg.m4]) > > > + > > Yann (Buildroot developer) found out, that this include > can be avoided when the aclocal.m4 would be moved. > It seems autoconf does not search for macros in any > global directories, if this file is in the current > directory found. Yes, I was wondering if there was a way to avoid the full path. > > Add this to configure.in: AC_CONFIG_MACRO_DIR([m4]) > mkdir m4 > git mv aclocal.m4 m4/autofs-macros.m4 > > Should I update the patch, according to this? You could, to save me a bit of time, I won't get to it until tomorrow (my time). Ian -- To unsubscribe from this list: send the line "unsubscribe autofs" in