Skip to content

haskell.compiler.ghc*Binary: fix globbing#260410

Merged
maralorn merged 1 commit intoNixOS:haskell-updatesfrom
ncfavier:ghc-fix-glob
Oct 22, 2023
Merged

haskell.compiler.ghc*Binary: fix globbing#260410
maralorn merged 1 commit intoNixOS:haskell-updatesfrom
ncfavier:ghc-fix-glob

Conversation

@ncfavier
Copy link
Copy Markdown
Member

Fix the --package-db="$out/lib/"ghc-*/package.conf.d glob (this does not do what you think!).

Disable nullglob as it seems incompatible with the way globs are used throughout those files, and caused some errors to be silently ignored.

@github-actions github-actions bot added the 6.topic: haskell General-purpose, statically typed, purely functional programming language label Oct 11, 2023
@ofborg ofborg bot added 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 5001+ This PR causes many rebuilds on Linux and must target the staging branches. labels Oct 11, 2023
@delroth delroth added 12.approvals: 1 This PR was reviewed and approved by one person. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages. labels Oct 13, 2023
Copy link
Copy Markdown
Member

@sternenseemann sternenseemann left a comment

Choose a reason for hiding this comment

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

We actually need to check for

  • "$out"/lib/ghc-*/lib/package.conf.d (hadrian) and
  • "$out"/lib/ghc-*/package.conf.d (make)

and use whichever exists, since there is a mix between what was used to build the bindists here.

@sternenseemann
Copy link
Copy Markdown
Member

sternenseemann commented Oct 16, 2023

Interesting that we have hadrian bindists in the earlier binary GHCs, but none of the hadrian specific of the 9.2.4 one. A mystery to ponder some other time… Thanks!

Probably makes sense to merge this big rebuild the next iteration around.

@maralorn
Copy link
Copy Markdown
Member

maralorn commented Oct 18, 2023

Am I correct in assuming that we merge this the next time we want to do a full set rebuild?

@maralorn maralorn merged commit fe83226 into NixOS:haskell-updates Oct 22, 2023
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: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 5001+ This PR causes many rebuilds on Linux and must target the staging branches. 12.approvals: 1 This PR was reviewed and approved by one person. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants