On 1/12/2015 2:25 AM, Andrew Haley wrote:
On 12/01/15 04:31, Dean Long wrote:
I found a small problem with the new config.sub wrapper.  It works with
the bash shell but not with the dash shell.
The problem seems to be with this line:

      result=`. $DIR/autoconf-config.sub $sub_args "$@"`

"dash" doesn't seem to support args passed with ".", so $sub_args "$@"
are ignored.
Thank you.  Sorry for the breakage; I didn't intend to use any
non-portable shell idioms.  I'm not expert enough to say whether this
is a bug in dash, but I don't suppose that matters: we are where we
are.

There's no reason not to replace the "." with "bash".  Do you want me
to do anything?

I guess not, since we only support bash. However, I did notice another problem with the
same file.  "aarch64-linux-gnu" comes out as "aarch64-linux-gnu" instead of
"aarch64-unknown-linux-gnu". I came up with a simpler version, where I replace "aarch64-" with "arm-", run autoconf-config.sub, then replace "arm-" back to "aarch64-".

dl

Andrew.

Reply via email to