Re: Spurious datarootdir warnings in Wine configure

[Ralf Wildenhues]

* Alexandre Julliard wrote on Wed, Jun 21, 2006 at 09:00:45PM CEST:
> Ralf Wildenhues <address@hidden> writes:
> > * Paul Eggert wrote on Tue, Jun 20, 2006 at 10:43:04PM CEST:
> >> OK, but the new test he added (marked '+' below):
> >> 
> >>    { ac_out=`sed -n '/\${datarootdir}/p' "$tmp/out"`; test -n "$ac_out"; } 
> >> &&
> >> +  { ac_out=`sed -n '/^[[  ]]*datarootdir[[        ]]*:*=/p' "$tmp/out"`; 
> >> test -z "$ac_out"; } &&
> >> 
> >> looks pretty portable to me.  If it breaks, then I'd expect the
> >> existing sed-based test to break too.
> >
> > Well.  This change causes us to issue a warning for an empty .in file.
> > So, apologies for rejecting it for the wrong reason.  ;-)
> 
> The change only adds an extra condition that must be fulfilled,

No.  The previous post-sed check tested for presence of literal
  ${datarootdir}

You test checks for _absence_ of something that resembles an
initialization of the `datarootdir' variable.

> so all
> it can do is silence the existing warning, I don't see how it can
> possibly cause an extra warning (and I certainly don't see one here
> with an empty .in file).

Apply your patch, rebuild and reinstall Autoconf, then:

cat >configure.ac <<EOF
AC_INIT
AC_CONFIG_FILES(foo)
AC_OUTPUT
EOF
touch foo.in
autoconf
./configure
| configure: creating ./config.status
| config.status: creating foo
| config.status: WARNING: foo contains a reference to the variable `datarootdir'
| which seems to be undefined.  Please make sure it is defined.

Cheers,
Ralf