Skip to content

Comments

doctest 0.20.1 is unbroken#196627

Closed
9999years wants to merge 1 commit intoNixOS:masterfrom
9999years:doctest-unbroken
Closed

doctest 0.20.1 is unbroken#196627
9999years wants to merge 1 commit intoNixOS:masterfrom
9999years:doctest-unbroken

Conversation

@9999years
Copy link
Contributor

Description of changes

Not sure how to actually get doctest 0.20.1 in, but I'm getting an error trying to use it downstream.

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 22.11 Release Notes (or backporting 22.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

@github-actions github-actions bot added the 6.topic: haskell General-purpose, statically typed, purely functional programming language label Oct 18, 2022
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Oct 18, 2022
Copy link
Member

Choose a reason for hiding this comment

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

These dontChecks were added to avoid depending on doctest, now that it is unbroken we should reenable tests and check if they can now be executed properly.

@sternenseemann
Copy link
Member

We'll get doctest 0.20.1 on the next regular hackage bump after which I'd be happy to merge this PR into haskell-updates. Until then you'll have to undo the stale overrides yourself.

@9999years
Copy link
Contributor Author

We'll get doctest 0.20.1 on the next regular hackage bump after which I'd be happy to merge this PR into haskell-updates. Until then you'll have to undo the stale overrides yourself.

Is there a way to do this without forking nixpkgs?

@sternenseemann
Copy link
Member

Reversing changes to derivations is hard, but you can replace the entire compiler specific overlay if you want to:

haskell.packages.ghc942.override {
  compilerConfig = { pkgs, haskellLib }: self: super: {};
}

What you probably want to do is reuse the existing overlay and remove stuff you don't want:

pkgs.haskell.packages.ghc942.override {
  compilerConfig = { pkgs, haskellLib }@args: self: super:
    builtins.removeAttrs (
      import (pkgs.path + "/pkgs/development/haskell-modules/configuration-ghc-9.4.x.nix") args self super
    ) [ "override-I-don't-like" ];
}

@9999years 9999years force-pushed the doctest-unbroken branch 2 times, most recently from 09c75bd to 358a1ff Compare October 20, 2022 20:03
@sternenseemann
Copy link
Member

Done in 26ec809.

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: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants