[SSSD] [PATCH] build: call AC_BUILD_AUX_DIR before anything else

Jakub Hrozek jhrozek at redhat.com
Thu Oct 30 13:52:49 UTC 2014


On Sun, Oct 12, 2014 at 06:05:47PM +0200, Jakub Hrozek wrote:
> On Sun, Oct 12, 2014 at 05:55:36PM +0200, Jakub Hrozek wrote:
> > On Sat, Oct 11, 2014 at 03:36:07PM +0200, Jan Engelhardt wrote:
> > > sssd's configure.ac (abridged) contains these lines:
> > > 
> > > 	AC_INIT([sssd], ...)
> > > 	m4_ifdef([AC_USE_SYSTEM_EXTENSIONS],
> > > 		[AC_USE_SYSTEM_EXTENSIONS], [AC_GNU_SOURCE])
> > > 	AC_CONFIG_AUX_DIR([build])
> > > 
> > > When turned into configure, this will be emitted:
> > > 
> > > 	ac_aux_dir=
> > > 	for ac_dir in build "$srcdir"/build; do
> > > 	  if test -f "$ac_dir/install-sh"; then
> > > 	    ac_aux_dir=$ac_dir
> > > 	    ac_install_sh="$ac_aux_dir/install-sh -c"
> > > 	    break
> > > 
> > > However, with automake commit v1.14.1-36-g7bc5927, this will be emitted
> > > instead:
> > > 
> > > 	ac_aux_dir=
> > > 	for ac_dir in "$srcdir" "$srcdir/.." "$srcdir/../.."; do
> > > 	  if test -f "$ac_dir/install-sh"; then
> > > 	    ac_aux_dir=$ac_dir
> > > 	    ac_install_sh="$ac_aux_dir/install-sh -c"
> > > 	    break
> > > 
> > > As configure no longer looks into build/ for install-sh, running
> > > ./configure fails:
> > > 
> > > 	configure: error: cannot find install-sh, install.sh,
> > > 	or shtool in "." "./.." "./../.."
> > > 
> > > I think the error is that someone placed AC_BUILD_AUX_DIR
> > > too late. Move it upwards.
> > 
> > Thank you for the patch and for the detailed explanation. I tested the
> > patch builds fine on F-20 and RHEL-6.
> > 
> > ACK
> 
> * master: 703dc1eb5b050b24235a6640f271d34ea008cf98

Per Timo Aaltonen's request, also pushed to sssd-1-11:
e6546e2703bb7725efc0446f99931b2858c61400



More information about the sssd-devel mailing list