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

ICU version mismatch #4

Closed
vinjana opened this issue Mar 1, 2018 · 2 comments
Closed

ICU version mismatch #4

vinjana opened this issue Mar 1, 2018 · 2 comments

Comments

@vinjana
Copy link

vinjana commented Mar 1, 2018

I have the following packages installed:

r-base=3.4.1=0
r-stringr=1.2.0=r3.4.1_0
icu=54.1

There are many more packages, but the important point is that some of them apparently depend on ICU 54, not 58. Then when I do load the stringi library in R I get:

> library(stringi)
Error: package or namespace load failed for 'stringi' in dyn.load(file, DLLpath = DLLpath, ...):
 unable to load shared object '/data/kensche/work/share/miniconda3/envs/test/lib/R/library/stringi/libs/stringi.so':
  libicui18n.so.58: cannot open shared object file: No such file or directory
  • Should ICU not be a runtime dependency of r-stringi to ensure such problems do not occur?
  • Is it possible to get a version linked against ICU 54.0, which is the version that the set of tools that I need currently supports?

I'd be grateful for your help!

@mariusvniekerk
Copy link
Member

Make a pull request thanks

@jdblischak
Copy link
Member

@vinjana Yes, you are correct. ICU should be a runtime dependency of r-stringi because it is listed as a system requirement of the package.

Please submit a PR with ICU 54 as a runtime dependency:

  - icu 54.*  # [not win]

Make sure to bump the build number from 0 to 1.

The reason to skip ICU on Windows is that ICU 54 was not built for Windows:

https://anaconda.org/anaconda/icu/files?version=54.1

Also, conda-forge has moved to pinning ICU to 58.*, so after we create a ICU 54 version, we should update the recipe to use ICU 58 (and also remove the [not win] since ICU 58 was built for Windows).

https://anaconda.org/conda-forge/icu/files?version=58.2
https://github.com/conda-forge/conda-forge-pinning-feedstock/blob/e206a9ea6d1d9a06deae0d3cac73b8768629f32f/recipe/conda_build_config.yaml#L96

What are the conda-forge packages you are using that are still pinned to ICU 54? These also need to be updated.

dfornika added a commit to dfornika/r-stringi-feedstock that referenced this issue Apr 27, 2018
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

No branches or pull requests

4 participants