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

opensuse42 + r36: install libicu from zypper #69

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pachadotdev
Copy link

see https://issues.apache.org/jira/browse/ARROW-12786
tldr; stringi shall try to build icu69 and fail, therefore installing from zypper eases installing stringr

@glin
Copy link
Collaborator

glin commented May 14, 2021

Hi, thanks for the PR and heads up. openSUSE 42 provides ICU 52 by default, while stringi requires ICU >= 55, so this unfortunately won't get stringi building again. libicu-devel is also already installed in these images, since we've made it a dependency of the R binaries (https://github.com/rstudio/r-builds/blob/ae1220091a564ee943be818f4146c990a80941cb/builder/package.opensuse-42#L51).

This is a tough issue, and we're considering two workarounds for the RStudio Package Manager binaries:

@pachadotdev
Copy link
Author

Hi, thanks for the PR and heads up. openSUSE 42 provides ICU 52 by default, while stringi requires ICU >= 55, so this unfortunately won't get stringi building again. libicu-devel is also already installed in these images, since we've made it a dependency of the R binaries (https://github.com/rstudio/r-builds/blob/ae1220091a564ee943be818f4146c990a80941cb/builder/package.opensuse-42#L51).

This is a tough issue, and we're considering two workarounds for the RStudio Package Manager binaries:

* Fall back to stringi's ICU 55 bundle by disabling stringi's C++11 support: https://stringi.gagolewski.com/install.html#c-11-issues. You should be able to set the `STRINGI_DISABLE_CXX11` environment variable to temporarily work around the ICU 69 compilation issue.

* Wait for the next CRAN release of stringi, or install the development version of stringi. The issue appears to now be resolved in the development version of stringi: [gagolews/stringi#431](https://github.com/gagolews/stringi/issues/431)

thanks a lot, I shall test with the development version
if I have a better solution, I shall edit my PR

@glin
Copy link
Collaborator

glin commented May 18, 2021

@pachamaltese It looks like stringi was updated on CRAN yesterday, and RSPM now has prebuilt binaries for it on openSUSE 42. Could you check if this resolves the stringr build issues?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants