-
Notifications
You must be signed in to change notification settings - Fork 237
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
haskell-nix.extraPkgconfigMappings #1667
haskell-nix.extraPkgconfigMappings #1667
Conversation
@@ -11,6 +11,8 @@ final: prev: { | |||
# overlays. | |||
defaultModules = []; | |||
|
|||
# TODO: doc etc | |||
extraPkgconfigMappings = {}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought making this an attribute of haskell-nix
and leveraging overlays would be the most composable route to propagating this to where it needed to go (similar to defaultModules
).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should probably be part of the project-common module.
I'm wondering if a better fix might be to just add these packages ( It used to be that there was an implicit mapping for all packages where the names matched (as is the case here). We had to loose that mapping with the new dummy I'll merge this feature though because there is a use case where multiple It might also be useful to have a way to exclude things from the mapping for example if:
|
bors try |
tryBuild succeeded: |
Neither library emits a But for now, I'll continue to use this. In general, users will likely run into unpackaged C libraries they wish to package themselves with |
Should we document that directly on the https://input-output-hk.github.io/haskell.nix/ manual? |
Fixes #1664
You can try the
sdl-gpu-minimal
example here https://gitlab.com/macaroni.dev/macaroni.nix/-/merge_requests/12It maps multiple new libraries and works on Linux and Windows (see #1666 for a change I had to make) currently.