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

Remove circular symlink from language-plutus-core #1970

Merged
merged 1 commit into from
Apr 15, 2020
Merged

Conversation

michaelpj
Copy link
Contributor

Tested and this works fine with:

  • cabal test
  • stack test
  • nix build

@michaelpj michaelpj requested a review from reactormonk April 14, 2020 17:50
@reactormonk
Copy link
Contributor

Option c) Add some TH magic so it can detect HIE. Takes too long IMO.

@michaelpj
Copy link
Contributor Author

I don't understand what goes wrong with HIE. Can you elaborate? If it doesn't build when it works with both stack and cabal that's IMO a bug in HIE.

@reactormonk
Copy link
Contributor

Yeah, it's a bug. I reported it here: fpco/th-utilities#10 May also be a HIE bug. But as bugs go, it's fairly easy to work around.

@michaelpj
Copy link
Contributor Author

Okay, well there's no point having the file-embed stuff then.

@michaelpj
Copy link
Contributor Author

Fixed.

@reactormonk
Copy link
Contributor

Okay, well there's no point having the file-embed stuff then.

It helps a tiny bit, but that's about it.

@michaelpj michaelpj merged commit 90d507b into master Apr 15, 2020
@reactormonk reactormonk deleted the fix/lpc-symlink branch April 16, 2020 10:39
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