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
buku: 3.8 -> 4.1 #54838
buku: 3.8 -> 4.1 #54838
Conversation
@GrahamcOfBorg build buku |
The build is failing. Why did you check the checkmark that you built and tested it with an enabled sandbox if that can't be true? I guess maybe it could be that you have an overlay that disables tests of everything, if so you should disable that for submitting PRs. |
@@ -1,14 +1,14 @@ | |||
{ stdenv, python3, fetchFromGitHub, fetchpatch }: |
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.
To circumvent the dateparser problem:
{ stdenv, python3, fetchFromGitHub, fetchpatch }: | |
{ stdenv, python36, fetchFromGitHub, fetchpatch }: |
and add a comment with the reason.
This should be done in a separate commit referencing #52766 in the commit message so that it can easily be reverted.
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.
Good idea. I tried it, but now I got an odd problem. The depending modules are still using python37!
❯ nix build -f . buku ~/nixpkgs
builder for '/nix/store/c30nlyv9wjwzizd7x37hvh4cmqn4k8ma-python3.7-dateparser-0.7.0.drv' failed with exit code 1; last 10 log lines:
- M
+ Miércoles 31 Diciembre 2014
----------------------------------------------------------------------
Ran 23161 tests in 101.451s
FAILED (failures=3, skipped=7)
Test failed: <unittest.runner.TextTestResult run=23161 errors=0 failures=3>
error: Test failed: <unittest.runner.TextTestResult run=23161 errors=0 failures=3>
cannot build derivation '/nix/store/dpvjy2zrn6px51j9kg64vrpnpi32ljgq-python3.7-maya-0.3.3.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/d1pmqfwddlw0627vgi2hli14168d6x5g-python3.7-Flask-Common-0.3.0.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/r6w3vdf98wbwd4b67x1cqbbj62d0c1lq-python3.7-httpbin-0.6.2.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/c24lh992njn2534iv57ykkmknii7wayf-python3.7-pytest-httpbin-0.3.0.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/p6lhmcprvvh6smy4hy3cnk7vr46lmlkw-python3.7-vcrpy-2.0.1.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/5jq53l30nic9ifsvx8y6l4giynhxk6x7-buku-4.1.drv': 1 dependencies couldn't be built
[0 built (1 failed)]
error: build of '/nix/store/5jq53l30nic9ifsvx8y6l4giynhxk6x7-buku-4.1.drv' failed
I have no clue how this can happen.
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.
That's not possible. It must have been a mistake on your side. Everything's working fine on my side.
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.
You are right, I built buku from my overlay and not the patched one.
I have to keep in mind to deactivate all overlays!
still not working, since depending python modules uses python37 instead of wanted python36
Now we still have
in the commit message :( |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)