Skip to content

ghcWithPackages: expose targetPrefix; tests.haskell.ghcWithPackages: verify -fllvm works with useLLVM#441471

Merged
sternenseemann merged 3 commits intoNixOS:masterfrom
sternenseemann:ghc-with-packages-use-llvm-test
Sep 11, 2025
Merged

ghcWithPackages: expose targetPrefix; tests.haskell.ghcWithPackages: verify -fllvm works with useLLVM#441471
sternenseemann merged 3 commits intoNixOS:masterfrom
sternenseemann:ghc-with-packages-use-llvm-test

Conversation

@sternenseemann
Copy link
Member

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux (cross)
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

Note that this test also succeeds in pkgsCross though using a stdenv
with cc is crucial, since its setup hooks are required for -lgmp to work.

Maybe this is something we should look at for GHC again, maybe we can
add extra entries to the package db, so it passes the necessary -L flags
by itself?
Obviously, we can't run GHCi of a cross-compiler, but respecting targetPrefix
will give a more helpful error message:

    <command line>: not built for interactive use
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 6.topic: haskell General-purpose, statically typed, purely functional programming language labels Sep 9, 2025
Copy link
Member

@emilazy emilazy left a comment

Choose a reason for hiding this comment

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

Haven’t tested; code looks fine.

@nixpkgs-ci nixpkgs-ci bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Sep 11, 2025
@sternenseemann sternenseemann added this pull request to the merge queue Sep 11, 2025
@sternenseemann
Copy link
Member Author

Why does the merge queue think No PR failures has not reported yet while having a green tick on the latest commit?!

Merged via the queue into NixOS:master with commit 7caff6e Sep 11, 2025
33 of 35 checks passed
@sternenseemann sternenseemann deleted the ghc-with-packages-use-llvm-test branch September 11, 2025 12:52
@wolfgangwalther
Copy link
Contributor

Because that's the "No PR failures" run for the PR workflow. The Merge Queue runs it again.

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 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants