Bug #14876
closed
/bin/sh: bad substitution since r63679 on Solaris
Description
r63679以降、Solaris 上にて 64ビットでコンパイルすると、
mjit_config.h 作成中に以下のように "/bin/sh: bad substitution" エラーになります。
32ビットコンパイルでは発生しません。（rubyci.orgのSolarisは32ビットのみ）
r63679 で変更された部分が原因と思われますが、シェルとmakeの変数が複雑に入り乱れているため切り分けができていません。
cc -errtags=yes -xO4 -xtarget=sparc64xplus -m64 -DRUBY_DEVEL=1 -DRUBY_EXPORT -DCANONICALIZATION_FOR_MATHN -I/usr/local/64/lib/libffi-3.0.10/include -I/usr/local/64/include -D_XOPEN_SOURCE=600 -I. -I.ext/include/sparc64-solaris2.10 -I./include -I. -I./enc/unicode/10.0.0 -o math.o -c math.c
+ echo #ifndef RUBY_MJIT_CONFIG_H
+ echo #define RUBY_MJIT_CONFIG_H 1
+ echo
sep=
+ quote MJIT_MIN_HEADER_NAME /include/ruby-2.6.0/sparc64-solaris2.10/rb_mjit_min_header-2.6.0.h
+ printf #define MJIT_MIN_HEADER_NAME
+ shift
+ printf "%s" /include/ruby-2.6.0/sparc64-solaris2.10/rb_mjit_min_header-2.6.0.h
+ echo
sep=,
+ quote MJIT_CC_COMMON cc
+ printf #define MJIT_CC_COMMON
+ shift
+ printf "%s", cc
+ echo
+ quote MJIT_CFLAGS MJIT_ARCHFLAG -w
+ printf #define MJIT_CFLAGS MJIT_ARCHFLAG
+ shift
+ printf "%s", -w
+ echo
+ quote MJIT_OPTFLAGS
+ printf #define MJIT_OPTFLAGS
+ shift
+ echo
+ quote MJIT_DEBUGFLAGS -g
+ printf #define MJIT_DEBUGFLAGS
+ shift
+ printf "%s", -g
+ echo
+ quote MJIT_LDSHARED cc -G
+ printf #define MJIT_LDSHARED
+ shift
+ printf "%s", cc -G
+ echo
+ quote MJIT_DLDFLAGS MJIT_ARCHFLAG -m64 -L/usr/local/64/lib -R/usr/local/64/lib
+ printf #define MJIT_DLDFLAGS MJIT_ARCHFLAG
+ shift
+ printf "%s", -m64 -L/usr/local/64/lib -R/usr/local/64/lib
+ echo
+ quote MJIT_LIBS -L/XXXXXXXXXX/lib
+ printf #define MJIT_LIBS
+ shift
+ printf "%s", -L/XXXXXXXXXX/lib
+ echo
+ echo #if 0
/bin/sh: bad substitution
make: *** [mjit_config.h] Error 1
Updated by ngoto (Naohisa Goto) almost 7 years ago
r63232 で追加された ${arch%=*} が "bad substitution" の原因でした。
前後の部分を引用します。
$${archs:+echo} $${archs:+'#if 0'}; \
for arch in $$archs; do \
echo "#elif defined __$${arch%=*}__"; \
quote " MJIT_ARCHFLAG " -arch $${arch%=*}; \
done; \
$${archs:+echo} $${archs:+'#endif'}; \
従来、archs に値が入っていなかったのでforループには突入しなかったのが、
r63679 の変更で archs に値が入るようになって、露見したようです。
Updated by ngoto (Naohisa Goto) almost 7 years ago
r63679 で Makefile.in の562行目に追加された archs="$${archs:-$(ARCH_FLAG:-arch=)}"; が悪さをしているようです。
この行は、Makefileの置換とシェルの置換が混ざっていて極めてわかりにくいのですが、
ARCH_FLAG="-arch i386 -arch x86_64" などが入っているとき「だけ」しか考慮されておらず、
ARCH_FLAG="-m64" や ARCH_FLAG="-march=i486" のときは想定外のようです。
ARCH_FLAG="-m64" のときは、mjit_config.h の末尾に以下が追加されることになります。
（シェルの ${arch%=*} の置換は正常に行われたと仮定）
#if 0
#elif defined __-m64__
#define MJIT_ARCHFLAG "-arch", "-m64",
#endif
この #elif defined __-m64__ というのはCのマクロとして明らかに異常と思いますし、
#define MJIT_ARCHFLAG "-arch", "-m64", というのも、 gcc のオプションとしてエラーになると思います。
Updated by ngoto (Naohisa Goto) almost 7 years ago
- Status changed from Open to Closed
Applied in changeset trunk|r63828.
Makefile.in: ARCH_FLAG may contain "-m64" etc.
- Makefile.in (mjit_config.h): ARCH_FLAG may contain "-m64", "-m32",
"-march=i486" and so on, but the change made with r63232 only
supports "-arch AAA -arch BBB" mainly used on Mac OS X.
To solve the issue, ARCH_FLAG is parsed and the architectures
specified in "-arch XXX" are added to $archs and the rest is
stored to $arch_flag. The $arch_flag is defined as MJIT_ARCHFLAG
if $archs is empty or the target architecture is not listed in $arch.
Fix build failure on Solaris 10 with ./configure CFLAGS="-m64".
[Bug #14876] [ruby-dev:50583]
Updated by ngoto (Naohisa Goto) over 6 years ago
- Related to Bug #15319: Since r65735, tool/mjit_archflag.sh related error on Solaris 10 added