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
jupyterhub, jupyter-telemetry: 1.0.0 -> 1.1.0, init at 0.1.0 #94866
Conversation
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.
should have two commits:
pythonPackages.jupyter-telemetry: init at 0.1.0
pythonPackages.jupyterhub: 1.0.0 -> 1.1.0
thanks for opening your first PR with nixpkgs :) |
related: #94783 |
Thanks for pointing this out! This will be huge for where I want to end up. |
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.
just realized this was made against release-20.03
please target master branch.
Backports to release branches only really happen if the application is unusable without the backport (E.g. Discord), or security patches.
Full explanation
When backporting changes, please follow https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#backporting-changes.
Namely, you should be doing git cherry-pick -x <rev>
from a commit that has already landed in master. If the branches have diverged, you may alter the commit or add another commit to ensure that the package is able to still evaluate and build.
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.
ERROR: Package 'jupyter-telemetry' requires a different Python: 2.7.18 not in '>=3.5'
builder for '/nix/store/27242kazi4zb78481lhgxfxvhhd4j590-python2.7-jupyter_telemetry-0.1.0.drv' failed with exit code 1
cannot build derivation '/nix/store/0dsj3sfaaxq1jwpnxgplnj5il1566r7k-env.drv': 1 dependencies couldn't be built
[10 built (1 failed), 271 copied (570.2 MiB), 101.9 MiB DL]
error: build of '/nix/store/0dsj3sfaaxq1jwpnxgplnj5il1566r7k-env.drv' failed
https://github.com/NixOS/nixpkgs/pull/94866
1 package failed to build:
python27Packages.jupyter-telemetry
|
||
buildPythonPackage rec { | ||
pname = "jupyter_telemetry"; | ||
version = "0.1.0"; |
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.
version = "0.1.0"; | |
version = "0.1.0"; | |
disabled = isPy27; |
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.
I noticed two things: https://github.com/jupyter/telemetry/blob/341f34421d73f11ec5538da8b04a52eb632f7511/setup.py#L33
and
disabled = pythonOlder "3.5"; |
It seems like these is more appropriate?
@@ -0,0 +1,28 @@ | |||
{ stdenv | |||
, buildPythonPackage | |||
, fetchPypi |
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.
, fetchPypi | |
, fetchPypi | |
, isPy27 |
Can do. I think I misunderstood the documentation when reading it. |
@chiroptical you didn't have to close, you could have done:
then in github, go to "edit" and change target branch to master |
Nice, I didn't realize I could do this. Will fix this up now. |
Corrected in #94885, sorry for my naivety. |
don't apologize for being unaware. I didn't know either until a few months ago :) |
Resolve issue with jupyterhub 1.0.0 through upgrade, requires jupyter-telemetry as a dependency
Motivation for this change
Experienced an issue with JupyterHub 1.0.0 which is resolved by upgrading to 1.1.0. During the upgrade, determined that jupyter-telemetry is a build dependency.
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)Unsure how to proceed (errors seem unrelated to change)
nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
fails with Python XML related errornix path-info -S
errors out