Re: [PATCH 2 of 2] companion_tools/autoconf: add autoconf tool

["Yann E. MORIN" <yann.morin.1998@anciens.enib.fr>]

Richard, All,

On Monday 11 January 2010 20:57:19 Richard Strand wrote:
> Yann E. MORIN wrote:
> > Directly call ./configure, do not use CT_CONFIG_SHELL. It's the
> > responsibility of ./configure to use CONFIG_SHELL (note there's
> > no leading CT_) internally. CONFIG_SHELL is correctly set and
> > exported.
> I know this is how it is supposed to work, but CONFIG_SHELL actually 
> seems to cause problems on my system.
> I am using Ubuntu 9.04 where /bin/sh points to dash and SHELL=/bin/bash. 
> When CONFIG_SHELL=/bin/bash then autoconf configure fails on my system 
> with the error "configure: error: no acceptable m4 could be found in 
> $PATH.". When CONFIG_SHELL is not set, or configure is explicitly called 
> with bash then this problem goes away!

OK. After all, we're not going to do heavy maintenace on those tools, so
it's a minor issue. But, please add a few lines commenting why you do it,
so it does not get tossed away in the future.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
`------------------------------^-------^------------------^--------------------'



--
For unsubscribe information see http://sourceware.org/lists.html#faq