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

haskellPackages: another round of mass-rebuilds #318363

Merged
merged 44 commits into from
Jun 20, 2024
Merged

Conversation

sternenseemann
Copy link
Member

This Merge

This PR is the regular merge of the haskell-updates branch into master.

This branch is being continually built and tested by hydra at https://hydra.nixos.org/jobset/nixpkgs/haskell-updates. You may be able to find an up-to-date Hydra build report at cdepillabout/nix-haskell-updates-status.

We roughly aim to merge these haskell-updates PRs at least once every two weeks. See the @NixOS/haskell team calendar for who is currently in charge of this branch.

haskellPackages Workflow Summary

Our workflow is currently described in pkgs/development/haskell-modules/HACKING.md.

The short version is this:

  • We regularly update the Stackage and Hackage pins on haskell-updates (normally at the beginning of a merge window).
  • The community fixes builds of Haskell packages on that branch.
  • We aim at at least one merge of haskell-updates into master every two weeks.
  • We only do the merge if the mergeable job is succeeding on hydra.
  • If a maintained package is still broken at the time of merge, we will only merge if the maintainer has been pinged 7 days in advance. (If you care about a Haskell package, become a maintainer!)

More information about Haskell packages in nixpkgs can be found in the nixpkgs manual.


This is the follow-up to #315167. Come to #haskell:nixos.org if you have any questions.

May be another one without a hackage/stackage bump? We'll see.

@sternenseemann
Copy link
Member Author

sternenseemann commented Jun 8, 2024

sternenseemann and others added 4 commits June 9, 2024 01:36
Those flags were not actually passed to GHC before, but to Setup.hs.

They were introduced in #86948. The related twitch live stream uses the
build of git-annex as a measurement. I get the following numbers when
building git-annex with doCheck = false:

 - for current master: 1:40 wall clock / 340s user
 - without any -A64M argument: 1:40 wall clock / 340s user
 - with this fix: 1:13 wall clock / 280s user

The idea was good, but the settings were never active.

More testing revealed that this seems to work on darwin just as well, so
we're removing the isLinux condition, too.
github-actions bot and others added 10 commits June 11, 2024 00:13
This is a left over to do from #308776. Rebasing existing PR (prior
to #308776) would need to be rebased on a change before the commit
included in this PR.
This refactor should simplify the code a little bit and make future
changes easier. I. e. for cross compiling GHC we'll have to update the
tools in the GHC settings file and calculate the host->target tool paths
for later use. Having a ready function for this will make this a lot
easier.
I took over maintainership of this package a while ago and fixed the
build errors in the latest hackage release
@sternenseemann

This comment was marked as outdated.

@sternenseemann
Copy link
Member Author

@alexfmpe

  • ghcjs-base and ghcjs-websockets are currently failing on all linux and darwin platforms
  • ghcjs-dom-hello is failing on aarch64-darwin

@maralorn
Copy link
Member

I think building ghcjs-base simply makes no sense on these plattforms does it?
Is that a result of removing the = null overrides from configuration-common?

@alexfmpe
Copy link
Member

alexfmpe commented Jun 19, 2024

ghcjs-dom-hello is failing on aarch64-darwin

I think it's not specific to the package and needs the cctools bump, see #316277 (comment)

I think building ghcjs-base simply makes no sense on these plattforms does it?
Is that a result of removing the = null overrides from configuration-common?

Correct

ghcjs-base and ghcjs-websockets are currently failing on all linux and darwin platforms

#321004

@sternenseemann
Copy link
Member Author

Some new regressions via master. tests.haskell (fixed by #321246) and webkit.

@sternenseemann sternenseemann merged commit 3f30378 into master Jun 20, 2024
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.