Skip to content
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

jupyterlab: Fix build #60159

Merged
merged 1 commit into from Apr 25, 2019
Merged

jupyterlab: Fix build #60159

merged 1 commit into from Apr 25, 2019

Conversation

matthiasbeyer
Copy link
Contributor

The build was broken by updating the underlying python library.
This is the quickfix.

Fixes: ca67a72 ("python37Packages.jupyterlab_server: 0.2.0 -> 0.3.0")

Closes #60154

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • other Linux distributions (centos7)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
  • Fits CONTRIBUTING.md.

The build was broken by updating the underlying python library.
This is the quickfix.

Fixes: ca67a72 ("python37Packages.jupyterlab_server: 0.2.0 -> 0.3.0")
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
@matthiasbeyer
Copy link
Contributor Author

matthiasbeyer commented Apr 24, 2019

@dotlambda and @tcosmo might be interested in this.

@tcosmo
Copy link

tcosmo commented Apr 24, 2019

Thank you!

Copy link
Member

@dotlambda dotlambda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The commit message should start with python3Packages.jupyterlab.

patches = [
(fetchpatch {
name = "Bump-jupyterlab_server-version";
url = "https://github.com/jupyterlab/jupyterlab/commit/3b8d451e6f9a4c609e60cde5fbb3cc84aae79951.patch";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
url = "https://github.com/jupyterlab/jupyterlab/commit/3b8d451e6f9a4c609e60cde5fbb3cc84aae79951.patch";
url = https://github.com/jupyterlab/jupyterlab/commit/3b8d451e6f9a4c609e60cde5fbb3cc84aae79951.patch;

@@ -22,6 +23,14 @@ buildPythonPackage rec {
"--set" "JUPYTERLAB_DIR" "$out/share/jupyter/lab"
];

patches = [
(fetchpatch {
name = "Bump-jupyterlab_server-version";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
name = "Bump-jupyterlab_server-version";
name = "bump-jupyterlab_server-version.patch";

@veprbl
Copy link
Member

veprbl commented Apr 24, 2019

@GrahamcOfBorg build python37Packages.jupyterlab

@matthiasbeyer
Copy link
Contributor Author

Rebased on latest master and squashed the commits. Ready for merge from my side.

@FRidh
Copy link
Member

FRidh commented Apr 25, 2019

@GrahamcOfBorg build python3.pkgs.jupyterlab

@FRidh FRidh merged commit dfd8f84 into NixOS:master Apr 25, 2019
@matthiasbeyer matthiasbeyer deleted the fix-jupyterlab branch April 25, 2019 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't install Jupyterlab with python 3.7
5 participants