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

Update pinned nixpgks #121

Closed
wants to merge 1 commit into from
Closed

Update pinned nixpgks #121

wants to merge 1 commit into from

Conversation

teggotic
Copy link

@teggotic teggotic commented Sep 17, 2023

I've tried to update pinned nixpkgs by rebasing @nh2 's commits to the latest master branch (as of 2023-09-16).

https://github.com/teggotic/nixpkgs/tree/static-haskell

@teggotic
Copy link
Author

Hey @nh2, how do I really verify/test that this really works.

I'm working on a tool, which now can be built with static version ghc946.

@nh2
Copy link
Owner

nh2 commented Sep 18, 2023

@teggotic

how do I really verify/test that this really works.

Build nix-build --no-link survey/default.nix -A working and -A workingStackageExecutables (defined here). I generally add the ones that fail to that exclusion list, so that workingStackageExecutables produces build errors.

You also have to update this:

# When changing this, also change the default version of Cabal declared below
compiler ? "ghc927",
# Tries to use `.a` files when evaluating TH, instead of `.so` files.
useArchiveFilesForTemplateHaskell ? false,
# See https://gitlab.haskell.org/ghc/ghc/-/wikis/commentary/libraries/version-history
defaultCabalPackageVersionComingWithGhc ?
({
ghc8107 = "Cabal_3_2_1_0";
ghc902 = "Cabal_3_4_1_0";
ghc927 = "Cabal_3_6_3_0";
ghc962 = "Cabal_3_10_1_0";
}."${compiler}"),

if the GHC version changes (e.g. if the nixpkgs default ghc changes from ghc927 to ghc928).

static-haskell-nix's master branch is generally based on the latest stable nixpkgs release. Given that this updates to nixpkgs master, we'd keep the PR open for now until that becomes NixOS 23.11, and then merge it.

Meanwhile, people who want to base on NixOS master can use the pin from this PR.


Ideally you would also update this line to the same commit as pinned by your submodule update:

else import (fetchTarball https://github.com/nh2/nixpkgs/archive/9e49f8f1f37bc906cda1adb33064c325d760819a.tar.gz) {}

@nh2 nh2 mentioned this pull request Jun 15, 2024
@nh2 nh2 closed this in #127 Jun 15, 2024
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