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

cabal2nix: resolve icu-i18n pkg-config module to icu #606

Closed
wants to merge 1 commit into from

Conversation

raehik
Copy link
Contributor

@raehik raehik commented Aug 14, 2023

The Nix package is icu, which provides three pkg-config modules including icu-i18n.

The Nix package is icu, which provides three pkg-config modules
including icu-i18n.
@raehik
Copy link
Contributor Author

raehik commented Aug 14, 2023

Question: Are the existing ICU pkg-config module entries correct? development/libraries/icu/base.nix states:

      pkgConfigModules = [
        "icu-i18n"
        "icu-io"
        "icu-uc"
      ];

So maybe we should swap out icui18n, icuuc etc. for icu-i18n, icu-uc. Or maybe we need both, if both are used?

@sternenseemann
Copy link
Member

Done in 0365d9b.

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.

2 participants