Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

configure scripts contain bashisms #139

Closed
atsampson opened this issue Oct 17, 2014 · 0 comments
Closed

configure scripts contain bashisms #139

atsampson opened this issue Oct 17, 2014 · 0 comments
Labels
Build A problem with building or installing the library.

Comments

@atsampson
Copy link

The bootstrap and configure scripts for ilmbase and openexr 2.2.0 make use of some non-standard shell features, so they work on a system where /bin/sh is bash, but not elsewhere. Most of these are test x == y where test x = y is intended; there's also one use of ${..//..} and one of arrays in ilmbase's "multiple OS X architectures" test.

The fixes are trivial: here's a patch for ilmbase and a patch for openexr.

@cary-ilm cary-ilm added the Build A problem with building or installing the library. label Jun 13, 2019
@cary-ilm cary-ilm added this to the Next Release milestone Jun 28, 2019
@cary-ilm cary-ilm added the TODO label Jun 28, 2019
kdt3rd added a commit to kdt3rd/openexr that referenced this issue Jul 21, 2019
Removes bash-isms from the autoconf bootstrap / configure.ac files

Signed-off-by: Kimball Thurston <kdt3rd@gmail.com>
@kdt3rd kdt3rd closed this as completed in ba329cb Jul 22, 2019
DominicJacksonBFX pushed a commit to boris-fx/mocha-openexr that referenced this issue Jun 22, 2022
Removes bash-isms from the autoconf bootstrap / configure.ac files

Signed-off-by: Kimball Thurston <kdt3rd@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build A problem with building or installing the library.
Projects
None yet
Development

No branches or pull requests

2 participants