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: add overrides for jsaddle projects #65048
Conversation
This resolves the Upstream prs: - ghcjs/jsaddle#100 - NixOS/nixpkgs#65048
96ddcf5
to
30a9da5
Compare
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.
You should make this change against the haskell-updates
branch instead
@@ -586,4 +586,10 @@ self: super: builtins.intersectAttrs super { | |||
# Tests require internet | |||
dhall_1_24_0 = dontCheck super.dhall_1_24_0; | |||
|
|||
jsaddle-wkwebview = overrideCabal super.jsaddle-wkwebview (old: { | |||
broken = false; |
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.
Remove jsaddle-wkwebview
from the list of broken packages in configuration-hackage2nix.nix
instead.
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.
Good to finally get this reflex-platform stuff in nixpkgs! @infinisil raises a good point but otherwise looks good.
CC @dmjio
I marked port to stable. It is harmless to fix broken packages on a release branch I think, and good shrink reflex-platform before the switch to 19.06. |
Note that webkitgtk stuff is still broken due to some weird version incompatibilities in the latest haskell packages update. Hopefully will resolve in a newer version. |
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.
Please address @infinisil's comments.
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)