Skip to content

haskellPackages.streamly-archive: unbreak#229443

Merged
maralorn merged 1 commit intoNixOS:haskell-updatesfrom
shlok:haskell-updates
May 2, 2023
Merged

haskellPackages.streamly-archive: unbreak#229443
maralorn merged 1 commit intoNixOS:haskell-updatesfrom
shlok:haskell-updates

Conversation

@shlok
Copy link
Copy Markdown
Contributor

@shlok shlok commented May 2, 2023

Things done
  • Tested compilation of all pkgs that depend on this change using nix-build --no-out-link -A haskellPackages.streamly-archive --arg config '{ allowBroken = true; }'
  • Ran maintainers/scripts/haskell/regenerate-hackage-packages.sh --fast and included the result in the PR. (Running this without --fast yielded a rm: cannot remove 'transitive-broken.ZwXjyKT': No such file or directory error, and resulted in a few changes to transitive-broken.yaml that had nothing to do with streamly-archive.)

@github-actions github-actions bot added the 6.topic: haskell General-purpose, statically typed, purely functional programming language label May 2, 2023
@maralorn
Copy link
Copy Markdown
Member

maralorn commented May 2, 2023

Thank you!

Sorry for the problems with the regenerate script. I am kind-of iterating on that one at the moment.

Can you add a comment that streamly-archive only works with streamly >= 0.9.0. The perfect solution would actually be an assert on the version of self.streamly.version < 0.9.0 so we know to remove this override.

Also, you might want to add yourself as maintainer of streamly-archive so that the package won’t break silently in the future.

@ofborg ofborg bot added 8.has: package (new) This PR adds a new package 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. labels May 2, 2023
@shlok
Copy link
Copy Markdown
Contributor Author

shlok commented May 2, 2023

Thanks! I have now made the suggested changes.

Edit: Adding myself as maintainer here: #229459

@maralorn
Copy link
Copy Markdown
Member

maralorn commented May 2, 2023

Excellent! Sadly, merging your other PR caused a merge conflict. Can you resolve that? Then I can merge this.

@shlok shlok closed this May 2, 2023
@shlok shlok force-pushed the haskell-updates branch from fb20bf5 to 2a950c2 Compare May 2, 2023 18:34
@github-actions github-actions bot removed the 6.topic: haskell General-purpose, statically typed, purely functional programming language label May 2, 2023
@shlok shlok reopened this May 2, 2023
@github-actions github-actions bot added the 6.topic: haskell General-purpose, statically typed, purely functional programming language label May 2, 2023
@shlok
Copy link
Copy Markdown
Contributor Author

shlok commented May 2, 2023

Sure, we can try again now.

@maralorn maralorn merged commit 68496b3 into NixOS:haskell-updates May 2, 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 8.has: package (new) This PR adds a new package 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants