search

gagolews / stringi

Fast and portable character string processing in R (with the Unicode ICU)
https://stringi.gagolewski.com/
Other
304 stars 44 forks source link

Require bash for configure #325

Closed xiamaz closed 5 years ago

xiamaz commented 6 years ago

The current autoconf generate configure file uses #!/bin/sh as the shebang, although the generated configure script will fail when run with dash symlinked to sh, which enforces POSIX sh much more strictly. Changing the shebang to /bin/bash fixes the issue.

gagolews commented 6 years ago

I don't think it's a good idea.

In particular bash extensions must not be used, and not all R platforms have a bash command, let alone one at /bin/bash.

see https://cran.r-project.org/doc/manuals/r-release/R-exts.html

xiamaz commented 6 years ago

But the autoconf script does not work using dash as the standard shell. If it is related to issues in POSIX compatibility, this should be considered a bug.

gagolews commented 6 years ago

this is generated by autoconf, possible solution is https://stackoverflow.com/questions/161064

I'll stick to what Writing R Extensions say about the location of bash