-
Notifications
You must be signed in to change notification settings - Fork 1
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
stringi build error #2
Comments
stringi news says
|
apparently ICU4C 70.1 is not good enough,
|
does not work even after installing ICU4C 74.1
running configure myself, and looking at config.log, I see:
|
hi @gagolews I'm trying to install stringi on a Linux server, and it was working fine until the CRAN update earlier this week.
with details in config.log shown in other comment above. (summary with first compilation error below)
could you please advise about how to fix? |
Is it possible that |
OK, I see that PKG_CPPFLAGS takes precedence over R's Makeconf CPPFLAGS, this should be reflected in the configure script as well. Hold on, I will try to fix it on the stringi side so that you do not have to, hopefully, change a thing. |
yes that seems to be the issue. config.log says
If I remove the conda include, then it works
It seems that this conda include is coming from the flags from this step
Are those flags coming from my build of ICU? Or from R?
which I thought meant that the bad conda include came from the flags that I used when I built ICU. |
Fixed - can you try the most recent development version of stringi (on github)? We need at least ICU v. 61, so of there is an older version on your system, it won't be used. We try fetching the flags for the system ICU via You can try inspecting the value of:
Also, try checking HTH |
thanks for the update. installing stringi from github seems to work,
|
for completeness it looks like PKG_CONFIG_PATH is now respected as well,
|
Splendid, cheers! |
Maybe need to upgrade ICU? ICU 70 is installed under $HOME (for redland), and ICU 58 is installed under $CONDA_PREFIX (for base R).
see https://stringi.gagolewski.com/install.html
The text was updated successfully, but these errors were encountered: