fix pcre config options. Required for future R lang releases#2561
fix pcre config options. Required for future R lang releases#2561boegel merged 1 commit intoeasybuilders:developfrom
Conversation
|
Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/6566/console for more details). This pull request is now ready for review/testing. Please try and find someone who can tackle this; contact @boegel if you're not sure what to do. |
| sources = [SOURCELOWER_TAR_GZ] | ||
|
|
||
| configopts = "--with-pic --disable-cpp --enable-utf8 --enable-unicode-properties" | ||
| configopts = "--with-pic --disable-cpp --enable-utf --enable-unicode-properties" |
There was a problem hiding this comment.
should enable both utf and utf8?
There was a problem hiding this comment.
--enable-utf8 is not needed. this is the output of ./configure --help
--enable-utf8 another name for --enable-utf. Kept only for
compatibility reasons
|
Test report by @boegel |
|
for the record...this fix is needed because when compiling R-devel version and loading the PCRE module in easybuild compiled without when recompiling PCRE with the In previous R release PCRE was included in the R tarball so PCRE was not a required dependency in the R easyconfig but in coming R version PCRE has to be included as a dependency because R tarball no longer includes it I did the tests with this R tarball: https://stat.ethz.ch/R/daily/R-devel_2016-02-23.tar.gz |
|
after recompiling PCRE with |
|
This is good to go imho, thanks @pescobar! |
fix pcre config options. Required for future R lang releases
No description provided.