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: unbreak typerep-map, co-log and co-log-sys #85486
Conversation
by the way, unbreaking by disabling tests is very mechanical and can be easily automated |
# Test suits dependencies are outdated | ||
typerep-map = dontCheck super.typerep-map; | ||
|
||
# Test suites are not implemented |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does this comment mean?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typerep-map's test suite failed because dependencies could not be satisfied (QuichCheck etc. are out-dated)
co-log-sys's test suite failed because it uses IO while also enables NoImplicitPrelude.
c2943c4
to
6d9922a
Compare
# Test suite's dependencies (QuickCheck etc.) are outdated | ||
typerep-map = dontCheck super.typerep-map; | ||
|
||
# Test suite is broken (uses IO while enables NoImplicitPrelude at the same time) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you report this problem upstream, and then link to it in this comment here in the code?
We'd like to easily be able to know when we can remove these types of overrides.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -1507,6 +1507,14 @@ self: super: { | |||
# Fixed at head, but hasn't cut a release in awhile. | |||
darcs = doJailbreak super.darcs; | |||
|
|||
# Test suite's dependencies (QuickCheck etc.) are outdated | |||
# see https://github.com/kowainik/typerep-map/issues/81 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This issue was resolved. New typerep-map-0.3.3.0
has been released on Hackage.
@@ -3637,8 +3637,6 @@ broken-packages: | |||
- cmt | |||
- cmv | |||
- cnc-spec-compiler | |||
- co-log |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Btw, newer updated versions of co-log-core
and co-log
are released to Hackage:
50d38e7
to
e567a3b
Compare
d8e9519
to
1d4449c
Compare
7418858
to
d990ff9
Compare
@poscat0x04 What's the status of this? Can it be closed, or are you still working on it? |
d990ff9
to
f617f24
Compare
Motivation for this change
Fix broken packages.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)