Skip to content

Conversation

angerman
Copy link
Collaborator

This will likely need to be split into a few PRs.

It does address #365 though.

@angerman angerman force-pushed the angerman/build-ghcjs branch from a99a094 to 6c68f7d Compare January 9, 2020 04:06
@angerman angerman marked this pull request as ready for review January 14, 2020 06:48
disassembler pushed a commit to IntersectMBO/cardano-prelude that referenced this pull request Feb 5, 2020
* temporarily removes from failing CI jobs
* adds required attribute with tests and lib for linux/darwin
* windows support will work with a haskell.nix bump when input-output-hk/haskell.nix#382 is on master
We will put it in a separate derivation and PR
…rman/build-ghcjs"

This reverts commit dbc8dea, reversing
changes made to 7b261be.

# Conflicts:
#	lib/call-cabal-project-to-nix.nix
Copy link
Collaborator Author

@angerman angerman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should comment:

 package.identifier.name == "distributive"

and revert the ghcjs-source change. That was only due to the double linking iirc.

We think the issues we had with duplicate code might have been caused
by the TH caching.
@angerman angerman merged commit f5b150c into master Feb 9, 2020
@angerman angerman deleted the angerman/build-ghcjs branch February 9, 2020 12:03
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.

3 participants