Skip to content

esqueleto: fix compilation with ghc-8.0.1#19331

Closed
vandenoever wants to merge 1 commit intoNixOS:release-16.09from
vandenoever:esqueleto-compile-fix-2
Closed

esqueleto: fix compilation with ghc-8.0.1#19331
vandenoever wants to merge 1 commit intoNixOS:release-16.09from
vandenoever:esqueleto-compile-fix-2

Conversation

@vandenoever
Copy link
Contributor

@vandenoever vandenoever commented Oct 7, 2016

Motivation for this change

#19247

patch from here: prowdsponsor/esqueleto#150

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • OS X
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@mention-bot
Copy link

@vandenoever, thanks for your PR! By analyzing the history of the files in this pull request, we identified @peti, @acowley and @bennofs to be potential reviewers.


# use master with patch that has build fixes
esqueleto = appendPatch (overrideCabal super.esqueleto (drv: {
src = pkgs.fetchFromGitHub {
Copy link
Member

Choose a reason for hiding this comment

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

This patch pins our version of esqueleto unconditionally to an more or less arbitrary git commit plus patches that upstream has not approved in any way. This kind of override is a maintenance nightmare because it will silently break updates in the main hackage-packages.nix file.

@peti
Copy link
Member

peti commented Oct 7, 2016

Dude, if you want to help improve the current situation, please help fix esqueleto UPSTREAM, i.e. on Hackage. Nixpkgs is not the right place to make these kind of intrusive changes. I find it somewhat annoying that it feels like I'm repeating the same point over and over again without achieving any observable affect.

@peti peti closed this Oct 7, 2016
@vandenoever
Copy link
Contributor Author

The same type of fix is present for structured-haskell-mode.
I do not want to take over maintainership of esqueleto.

@peti
Copy link
Member

peti commented Oct 7, 2016

The same type of fix is present for structured-haskell-mode.

We don't have any alternative there because structured-haskell-mode is outright broken with Emacs 25.x, which is the only version we have, so we need a git snapshot to have anything that's usable. esqueleto, on the other hand, compiles and works just fine with appropriate versions of persistent, which we do have in Nixpkgs.

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