RE: cygwin wrapper aclocal doesn't handle multiple -I options ?

RE: cygwin wrapper aclocal doesn't handle multiple -I options ?

[Rapp, Perry]

If I alter 
  /usr/bin/aclocal
from this

    -I )
       test $# = 1 && eval "$exit_missing_arg"
       shift
       opt_i="-I $1"
       shift ;;

to this

    -I )
       test $# = 1 && eval "$exit_missing_arg"
       shift
       opt_i=$opt_i" -I $1"
       shift ;;


then my use of multiple includes seems to work correctly with aclocal


Cordially,

Perry


--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Bug reporting:         http://cygwin.com/bugs.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/

Re: cygwin wrapper aclocal doesn't handle multiple -I options ?

[Charles Wilson]

Noted.  I need to release a new autoconf wrapper soon (autoconf-2.53a 
added the "-I" option to replace "-macrodir", so I need to handle that). 
  I'll do a new automake wrapper at the same time.

--Chuck


Rapp, Perry wrote:

> If I alter 
>   /usr/bin/aclocal
> from this
> 
>     -I )
>        test $# = 1 && eval "$exit_missing_arg"
>        shift
>        opt_i="-I $1"
>        shift ;;
> 
> to this
> 
>     -I )
>        test $# = 1 && eval "$exit_missing_arg"
>        shift
>        opt_i=$opt_i" -I $1"
>        shift ;;
> 
> 
> then my use of multiple includes seems to work correctly with aclocal
> 
> 
> Cordially,
> 
> Perry
> 
> 
> --
> Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
> Bug reporting:         http://cygwin.com/bugs.html
> Documentation:         http://cygwin.com/docs.html
> FAQ:                   http://cygwin.com/faq/
> 
> 
> 



--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Bug reporting:         http://cygwin.com/bugs.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/

RE: cygwin wrapper aclocal doesn't handle multiple -I options ?

[Bernard A Badger]

You should allow file names with spaces in them.
Use:
	opt_i="$opt_i -I '$1'"

BTW, a better name would be opt_I, not opt_i, since the option is -I, not -i.

<<Further nitpicks suppressed.>>

> -----Original Message-----
> From: cygwin-owner@cygwin.com [mailto:cygwin-owner@cygwin.com]On Behalf
> Of Charles Wilson
> Sent: Tuesday, June 25, 2002 11:04 PM
> To: Rapp, Perry
> Cc: cygwin@cygwin.com
> Subject: Re: cygwin wrapper aclocal doesn't handle multiple -I options ?
> 
> 
> Noted.  I need to release a new autoconf wrapper soon (autoconf-2.53a 
> added the "-I" option to replace "-macrodir", so I need to handle that). 
>   I'll do a new automake wrapper at the same time.
> 
> --Chuck
> 
> 
> Rapp, Perry wrote:
> 
> > If I alter 
> >   /usr/bin/aclocal
> > from this
> > 
> >     -I )
> >        test $# = 1 && eval "$exit_missing_arg"
> >        shift
> >        opt_i="-I $1"
> >        shift ;;
> > 
> > to this
> > 
> >     -I )
> >        test $# = 1 && eval "$exit_missing_arg"
> >        shift
> >        opt_i=$opt_i" -I $1"
> >        shift ;;
> > 
> > 
> > then my use of multiple includes seems to work correctly with aclocal
> > 
> > 
> > Cordially,
> > 
> > Perry
> > 
> > 
> > --
> > Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
> > Bug reporting:         http://cygwin.com/bugs.html
> > Documentation:         http://cygwin.com/docs.html
> > FAQ:                   http://cygwin.com/faq/
> > 
> > 
> > 
> 
> 
> 
> --
> Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
> Bug reporting:         http://cygwin.com/bugs.html
> Documentation:         http://cygwin.com/docs.html
> FAQ:                   http://cygwin.com/faq/

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Bug reporting:         http://cygwin.com/bugs.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/

Re: cygwin wrapper aclocal doesn't handle multiple -I options ?

[Charles Wilson]

Bernard A Badger wrote:

> You should allow file names with spaces in them.
> Use:
> 	opt_i="$opt_i -I '$1'"
> 
> BTW, a better name would be opt_I, not opt_i, since the option is -I, not -i.
> 
>


It's too late for this release of the wrappers, but patches gratefully 
accepted for the future.  However, your suggestion won't work:

"$opt_i -I '$1'"

using single quotes prevents $1 from being expanded at all.  What you'd 
need to do is something like:

"$opt_i -I \"$1\""

But I'm not sure if those doublequotes will be safe after multiple 
re-expansions as $opt_i is built and extended.  This needs some testing.

--Chuck




--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Bug reporting:         http://cygwin.com/bugs.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/

cygwin wrapper aclocal doesn't handle multiple -I options ?

[Rapp, Perry]

I have a hypothesis that the aclocal wrapper doesn't handle
multiple -I options, or at least doesn't pass them all to the
real aclocal.


If I replace my call

  aclocal -I m4 -I build/autotools

with

  /usr/autotools/devel/aclocal -I m4 -I build/autotools

then things work (it finds gettext, which is in ./m4/gettext.m4).

I looked at 

  /usr/bin/aclocal

and this section:



    -I )
       test $# = 1 && eval "$exit_missing_arg"
       shift
       opt_i="-I $1"
       shift ;;


suggests to me that it is replacing the earlier -I option with the
later ?

I am guessing a solution would be to do an append there,
but I'm no good at shell programming -- anyone know the
proper syntax ?

Cordially,

Perry



ps:



$ aclocal --version
aclocal (GNU automake) 1.6.1

Copyright 2002 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is
NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR
PURPOSE.

Written by Tom Tromey <tromey@redhat.com>


--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Bug reporting:         http://cygwin.com/bugs.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/