bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [gnulib PATCH]: new warning from ar on rawhide systems


From: Pavel Raiskup
Subject: Re: [gnulib PATCH]: new warning from ar on rawhide systems
Date: Thu, 02 Jul 2015 14:32:23 +0200
User-agent: KMail/4.14.9 (Linux/4.0.6-200.fc21.x86_64+debug; KDE/4.14.9; x86_64; ; )

On Wednesday 01 of July 2015 16:38:57 Pádraig Brady wrote:
> On 01/07/15 16:22, Pavel Raiskup wrote:
> > Yes, there is still nothing like that in gnulib-tool, am I right?  I mean
> > dependency among configure.ac-early snippets -- those seem to be all just
> > sorted by name (by 'LANG=C sort -u').
> > 
> > Would you consider something like this as acceptable?
> > 
> > diff --git a/gnulib-tool b/gnulib-tool
> > index ec82f35..f5bddbb 100755
> > --- a/gnulib-tool
> > +++ b/gnulib-tool
> > @@ -5188,6 +5188,10 @@ s,//*$,/,'
> >      echo "  m4_pattern_allow([^gl_ES\$])dnl a valid locale name"
> >      echo "  m4_pattern_allow([^gl_LIBOBJS\$])dnl a variable"
> >      echo "  m4_pattern_allow([^gl_LTLIBOBJS\$])dnl a variable"
> > +
> > +    echo "$final_modules" | grep "^extensions$" >/dev/null \
> > +        && echo "  AC_REQUIRE([gl_USE_SYSTEM_EXTENSIONS])"
> > +
> >      echo "  AC_REQUIRE([gl_PROG_AR_RANLIB])"
> >      if test -n "$uses_subdirs"; then
> >        echo "  AC_REQUIRE([AM_PROG_CC_C_O])"
> > 
> > I don't like that TBH, but if we planned to make the gl_PROG_AR_RANLIB
> > part of some module (and add some ordering into early snippets), this
> > would be enough for the meantime.
> 
> Yes that's a bit coupled but should work.
> I'll try to test later.

Thanks, just to make it complete I'm attaching real patch.

Pavel

Attachment: 0001-gnulib-common.m4-fix-gl_PROG_AR_RANLIB-AM_PROG_AR-cl.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]