Skip to content

ghcide: include and configure additional specific package versions#76103

Closed
mdorman wants to merge 19 commits intoNixOS:haskell-updatesfrom
mdorman:ghcide
Closed

ghcide: include and configure additional specific package versions#76103
mdorman wants to merge 19 commits intoNixOS:haskell-updatesfrom
mdorman:ghcide

Conversation

@mdorman
Copy link
Copy Markdown
Contributor

@mdorman mdorman commented Dec 20, 2019

Motivation for this change

In addition to the haskell-lsp and haskell-lsp-types, ghcide is tied
to particular versions of lsp-test, hie-bios and shake, so we need to
keep them around if it is to work.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.
Notify maintainers

cc @peti

cdepillabout and others added 19 commits December 18, 2019 11:06
This update was generated by hackage2nix v2.15.0-9-g650d530 from Hackage revision
commercialhaskell/all-cabal-hashes@dc5b0e7.
This update was generated by hackage2nix v2.15.0-9-g650d530 from Hackage revision
commercialhaskell/all-cabal-hashes@b2a082b.
haskellPackages.glirc is a "Console IRC Client." I've added a doJailbreak
to fix the build (thanks @infinisil) and added it to top-level/all-packages.nix
so people can find and install it as they would normally.

Would be nice to make this build in a way that allows the OTR extension to be
enabled. One thing at a time....
This update was generated by hackage2nix v2.15.0-11-g2fb6f7e from Hackage revision
commercialhaskell/all-cabal-hashes@4fa8739.
haskellPackages.snap-server: patch for network >= 3.0
spago: add update script for automatically updating to new versions
This PR fixes dhall_1_28_0, dhall-bash_1_0_25, and dhall-json_1_6_0 so
they build.

They all require a newer version of prettyprinter than we get from the
LTS package set.

This is from NixOS#75931 by @ijaketak.

Co-authored-by: Keito Kajitani <ijaketak@gmail.com>
haskellPackages.dhall_1_28_0: fix overrides so it builds
This adds dhall-1.27.0 to extra-packages in
configuration-hackage2nix.yaml.

dhall-1.27.0 is used by spago, so this package needs to be available.
…l_1_27_0

haskellPackages.dhall_1_27_0: generate this package
haskell-lsp: Added 0.18.* to extra-packages for ghcide
haskellPackages.postgresql-simple-url: mark unbroken
In addition to the haskell-lsp and haskell-lsp-types, ghcide is tied
to particular versions of lsp-test, hie-bios and shake, so we need to
keep them around if it is to work.
@cdepillabout
Copy link
Copy Markdown
Member

cc @turion and @mgttlinger who have also taken a look at this.

@cdepillabout cdepillabout added the 6.topic: haskell General-purpose, statically typed, purely functional programming language label Dec 21, 2019
@peti
Copy link
Copy Markdown
Member

peti commented Dec 26, 2019

Personally, I believe that these extensive overrides in Nixpkgs are a step in the wrong direction. I'd much rather see that we cooperate with upstream to make sure that ghcide compiles with recent versions of its dependencies.

@peti peti force-pushed the haskell-updates branch 2 times, most recently from 1f7024e to 7d2cc64 Compare December 26, 2019 12:07
@peti peti mentioned this pull request Dec 27, 2019
3 tasks
@peti peti closed this in 4d403a3 Dec 27, 2019
@mdorman mdorman deleted the ghcide branch March 5, 2023 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: haskell General-purpose, statically typed, purely functional programming language

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants