Skip to content

haskellPackages.fft: unmark broken#233884

Closed
SuperSandro2000 wants to merge 1 commit intoNixOS:haskell-updatesfrom
SuperSandro2000:haskell-fft
Closed

haskellPackages.fft: unmark broken#233884
SuperSandro2000 wants to merge 1 commit intoNixOS:haskell-updatesfrom
SuperSandro2000:haskell-fft

Conversation

@SuperSandro2000
Copy link
Member

Description of changes
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/)
  • 23.05 Release Notes (or backporting 22.11 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
  • Fits CONTRIBUTING.md.

@github-actions github-actions bot added the 6.topic: haskell General-purpose, statically typed, purely functional programming language label May 24, 2023
@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 May 24, 2023
Copy link
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.

Please run ./maintainers/scripts/haskell/regenerate-hackage-packages.sh to regenerate the package set—your manual edit is not quite correct.

It's probably easier to target haskell-updates with this, 23.05 needs a backport anyways.

@SuperSandro2000 SuperSandro2000 changed the base branch from master to haskell-updates June 6, 2023 21:42
@SuperSandro2000
Copy link
Member Author

done and rebased onto haskell-updates

@cdepillabout
Copy link
Member

@ofborg build haskellPackages.fft

@cdepillabout
Copy link
Member

@SuperSandro2000 It looks like the test suite failed for fft on x86_64-linux in ofborg:

https://github.com/NixOS/nixpkgs/pull/233884/checks?check_run_id=14058426877

I wonder if this is an expected error? (In which case, you could mark the package as dontCheck and create an issue upstream about the incorrect error.)

@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-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. and removed 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 Jun 7, 2023
@SuperSandro2000
Copy link
Member Author

SuperSandro2000 commented Jun 7, 2023

I can reproduce that when merging this branch into haskell-updates, building directly on this branch works. There is no error so I am not sure what I should tell upstream

Contents of fft-0.1.8.7-test-fft.log:

Float
dft idem 1D              : +++ OK, passed 1000 tests.
dftRC/CR idem  1D        : +++ OK, passed 1000 tests.
dftRC dft 1D             : +++ OK, passed 1000 tests.
dht idem 1D              : +++ OK, passed 1000 tests.
dft idem 2D              : +++ OK, passed 1000 tests.
dft idem 2D/2            : +++ OK, passed 1000 tests.
dft idem 2D/2'           : +++ OK, passed 1000 tests.
dftRC/CR idem  2D        : +++ OK, passed 1000 tests.
dftRC dft 2D             : +++ OK, passed 1000 tests.
dftRC dft 2D/2           : +++ OK, passed 1000 tests.
dht idem 2D              : +++ OK, passed 1000 tests.
dft idem 3D              : +++ OK, passed 1000 tests.
dft idem 3D/2            : +++ OK, passed 1000 tests.
dft idem 3D/2'           : +++ OK, passed 1000 tests.
dft idem 3D/3            : +++ OK, passed 1000 tests.
dft idem 3D/3'           : +++ OK, passed 1000 tests.
dft idem 3D/3''          : +++ OK, passed 1000 tests.
Double
dft idem 1D              : +++ OK, passed 1000 tests.
dftRC/CR idem  1D        : +++ OK, passed 1000 tests.
dftRC dft 1D             : +++ OK, passed 1000 tests.
dht idem 1D              : +++ OK, passed 1000 tests.
dft idem 2D              : +++ OK, passed 1000 tests.
dft idem 2D/2            : +++ OK, passed 1000 tests.
dft idem 2D/2'           : +++ OK, passed 1000 tests.
dftRC/CR idem  2D        : +++ OK, passed 1000 tests.
dftRC dft 2D             : +++ OK, passed 1000 tests.
dftRC dft 2D/2           : +++ OK, passed 1000 tests.
dht idem 2D              : +++ OK, passed 1000 tests.
Test suite test-fft: FAIL
Test suite logged to: dist/test/fft-0.1.8.7-test-fft.log

@cdepillabout
Copy link
Member

cdepillabout commented Jun 8, 2023

@SuperSandro2000 Thanks for reporting this.

I went ahead and marked it unbroken and disabled the test suite in fc8620b in #236431.

My guess is that the tests have recently started failing because of nick8325/quickcheck#359, but it is just a problem in the fft test suite. However, I wasn't able to find an issue tracker for fft, so I couldn't even find anywhere to report it :-|

@SuperSandro2000
Copy link
Member Author

Thanks!

@SuperSandro2000 SuperSandro2000 deleted the haskell-fft branch June 14, 2023 13:57
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-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants