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
deeptools,tebreak,python3Packages.cnvkit,python3Packages.HTSeq: Fix test #102140
Conversation
f390ce4
to
ff9c6fa
Compare
ff9c6fa
to
94dcd5a
Compare
94dcd5a
to
d4e0df3
Compare
Result of 7 packages failed to build:
Building currently fails because tests in pysam fail which should be resolved with #100814. |
d4e0df3
to
39a8cea
Compare
The PR is merged now. |
@SuperSandro2000 instead of disabling tests altogether, can we disable only the failing ones? |
If I did no mistake they all have zero tests indicated by |
39a8cea
to
3df68d3
Compare
That depends on how the tests should be run. |
The pypi tar balls often don't contain tests. I switch to the GitHub ones. We should really pay more attention to this in the future when updating/adding packages. |
Result of 7 packages marked as broken and skipped:
Failed because xmlsec failed to build. Edit: patching xmlsec to be reproducible and build in sandbox. |
3df68d3
to
3762141
Compare
Result of 1 package marked as broken and skipped:
5 packages failed to build:
8 packages built:
|
3762141
to
9c029bb
Compare
Result of 1 package marked as broken and skipped:
5 packages failed to build:
9 packages built:
I just build this locally... |
9c029bb
to
3faa754
Compare
Result of 1 package marked as broken and skipped:
4 packages failed to build:
10 packages built:
Only 3 more tries and this should build. |
3faa754
to
40f5bc7
Compare
40f5bc7
to
911f0bc
Compare
Result of 9 packages marked as broken and skipped:
1 package failed to build:
5 packages built:
|
Result of 7 packages built:
|
Result of 1 package marked as broken and skipped:
14 packages built:
|
1 package failed to build:
13 packages built:
The build failure seems unrelated to this pull request so LGTM on |
Result of 1 package failed to build:- kmymoney 13 packages built:- deeptools - gnucash - lasso - matrix-synapse - python27Packages.HTSeq - python37Packages.HTSeq - python37Packages.cnvkit - python37Packages.pysaml2 - python38Packages.HTSeq - python38Packages.cnvkit - python38Packages.pysaml2 - tebreak - xmlsec |
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.
diff LGTM
https://github.com/NixOS/nixpkgs/pull/102140
1 package marked as broken and skipped:
kmymoney
13 packages built:
deeptools gnucash lasso matrix-synapse python27Packages.HTSeq python37Packages.HTSeq python37Packages.cnvkit python37Packages.pysaml2 python38Packages.HTSeq python38Packages.cnvkit python38Packages.pysaml2 tebreak xmlsec
a497e9a
to
84f0c4b
Compare
Result of 14 packages built:
|
Result of 9 packages marked as broken and skipped:
6 packages built:
|
Motivation for this change
#100814
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)