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

pythonPackages.jupyterhub-ldapauthenticator: init at 1.1 #35467

Merged
merged 1 commit into from Feb 26, 2018
Merged

pythonPackages.jupyterhub-ldapauthenticator: init at 1.1 #35467

merged 1 commit into from Feb 26, 2018

Conversation

ixxie
Copy link
Contributor

@ixxie ixxie commented Feb 24, 2018

  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@ixxie ixxie requested a review from FRidh as a code owner February 24, 2018 12:18
@ixxie ixxie changed the title pythonPackages.jupyterhub-oauthenticator: init at 1.1 pythonPackages.jupyterhub-ldapauthenticator: init at 1.1 Feb 24, 2018

propagatedBuildInputs = [ jupyterhub ldap3 ];

disabled = pythonOlder "3.4";
Copy link
Member

Choose a reason for hiding this comment

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

Why that? The README says

I've only tested with python3 - anyone willing to test with python2 is welcome to do so! There's no reason it shouldn't work.

sha256 = "19dz3a3122wln8lkixj5jbh9x3cqlrcb3p7a53825cj72cmpcxwz";
};

# No tests in archive
Copy link
Member

Choose a reason for hiding this comment

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

There aren't even any on GitHub, so this should say something like No tests implemented

@@ -0,0 +1,32 @@

Copy link
Member

Choose a reason for hiding this comment

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

whitespace

@ixxie
Copy link
Contributor Author

ixxie commented Feb 24, 2018

@dotlambda - whoopsy, been a little sloppy there! Fixed now, hopefully.

@dotlambda
Copy link
Member

@GrahamcOfBorg build python2.pkgs.jupyterhub-ldapauthenticator python3.pkgs.jupyterhub-ldapauthenticator

@GrahamcOfBorg
Copy link

Failure on x86_64-linux (full log)

Partial log (click to expand)

while evaluating ‘getOutput’ at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/ogden/lib/attrsets.nix:450:23, called from undefined position:
while evaluating anonymous function at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/ogden/pkgs/stdenv/generic/make-derivation.nix:111:17, called from undefined position:
while evaluating ‘callPackageWith’ at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/ogden/lib/customisation.nix:113:35, called from /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/ogden/pkgs/top-level/python-packages.nix:9304:16:
while evaluating ‘makeOverridable’ at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/ogden/lib/customisation.nix:72:24, called from /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/ogden/lib/customisation.nix:117:8:
while evaluating anonymous function at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/ogden/pkgs/development/python-modules/jupyterhub/default.nix:1:1, called from /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/ogden/lib/customisation.nix:74:12:
while evaluating ‘makeOverridablePythonPackage’ at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/ogden/pkgs/top-level/python-packages.nix:40:37, called from /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/ogden/pkgs/development/python-modules/jupyterhub/default.nix:51:1:
while evaluating ‘makeOverridable’ at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/ogden/lib/customisation.nix:72:24, called from /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/ogden/pkgs/top-level/python-packages.nix:42:12:
while evaluating anonymous function at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/ogden/pkgs/development/interpreters/python/build-python-package.nix:26:1, called from /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/ogden/lib/customisation.nix:74:12:
while evaluating anonymous function at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/ogden/pkgs/development/interpreters/python/mk-python-derivation.nix:14:1, called from /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/ogden/pkgs/development/interpreters/python/build-python-package.nix:43:4:
jupyterhub-0.8.1 not supported for interpreter python2.7

@GrahamcOfBorg
Copy link

Failure on aarch64-linux (full log)

Partial log (click to expand)

while evaluating 'getOutput' at �[1m/var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/lib/attrsets.nix�[0m:450:23, called from undefined position:
while evaluating anonymous function at �[1m/var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/pkgs/stdenv/generic/make-derivation.nix�[0m:111:17, called from undefined position:
while evaluating 'callPackageWith' at �[1m/var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/lib/customisation.nix�[0m:113:35, called from �[1m/var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/pkgs/top-level/python-packages.nix�[0m:9304:16:
while evaluating 'makeOverridable' at �[1m/var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/lib/customisation.nix�[0m:72:24, called from �[1m/var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/lib/customisation.nix�[0m:117:8:
while evaluating anonymous function at �[1m/var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/pkgs/development/python-modules/jupyterhub/default.nix�[0m:1:1, called from �[1m/var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/lib/customisation.nix�[0m:74:12:
while evaluating 'makeOverridablePythonPackage' at �[1m/var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/pkgs/top-level/python-packages.nix�[0m:40:37, called from �[1m/var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/pkgs/development/python-modules/jupyterhub/default.nix�[0m:51:1:
while evaluating 'makeOverridable' at �[1m/var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/lib/customisation.nix�[0m:72:24, called from �[1m/var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/pkgs/top-level/python-packages.nix�[0m:42:12:
while evaluating anonymous function at �[1m/var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/pkgs/development/interpreters/python/build-python-package.nix�[0m:26:1, called from �[1m/var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/lib/customisation.nix�[0m:74:12:
while evaluating anonymous function at �[1m/var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/pkgs/development/interpreters/python/mk-python-derivation.nix�[0m:14:1, called from �[1m/var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/pkgs/development/interpreters/python/build-python-package.nix�[0m:43:4:
jupyterhub-0.8.1 not supported for interpreter python2.7

@dotlambda
Copy link
Member

dotlambda commented Feb 25, 2018

@GrahamcOfBorg build python3.pkgs.jupyterhub-ldapauthenticator

@ixxie You might want to add disabled = !isPy3k.

@GrahamcOfBorg
Copy link

Failure on x86_64-linux (full log)

Partial log (click to expand)

error: attribute ‘python3’ in selection path ‘python3.pkgs.python3.pkgs.jupyterhub-ldapauthenticator’ not found

@GrahamcOfBorg
Copy link

Failure on aarch64-linux (full log)

Partial log (click to expand)

�[31;1merror:�[0m attribute 'python3' in selection path 'python3.pkgs.python3.pkgs.jupyterhub-ldapauthenticator' not found

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Partial log (click to expand)

Installing collected packages: jupyterhub-ldapauthenticator
Successfully installed jupyterhub-ldapauthenticator-1.1
/tmp/nix-build-python3.6-jupyterhub-ldapauthenticator-1.1.drv-0/jupyterhub-ldapauthenticator-1.1
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/s708qhd4ryxiz0azlsjpab52yk43z3n2-python3.6-jupyterhub-ldapauthenticator-1.1
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/s708qhd4ryxiz0azlsjpab52yk43z3n2-python3.6-jupyterhub-ldapauthenticator-1.1/lib 
patching script interpreter paths in /nix/store/s708qhd4ryxiz0azlsjpab52yk43z3n2-python3.6-jupyterhub-ldapauthenticator-1.1
checking for references to /tmp/nix-build-python3.6-jupyterhub-ldapauthenticator-1.1.drv-0 in /nix/store/s708qhd4ryxiz0azlsjpab52yk43z3n2-python3.6-jupyterhub-ldapauthenticator-1.1...
/nix/store/s708qhd4ryxiz0azlsjpab52yk43z3n2-python3.6-jupyterhub-ldapauthenticator-1.1

propagatedBuildInputs = [ jupyterhub ldap3 ];

meta = with lib; {
description = "Simple LDAP Authenticator Plugin for JupyterHub.";
Copy link
Member

Choose a reason for hiding this comment

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

No .

, buildPythonPackage
, jupyterhub
, ldap3
, pythonOlder
Copy link
Member

Choose a reason for hiding this comment

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

Isn't used

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Partial log (click to expand)

Installing collected packages: jupyterhub-ldapauthenticator
Successfully installed jupyterhub-ldapauthenticator-1.1
/build/jupyterhub-ldapauthenticator-1.1
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/a95dawf4zpxfmy9qcdlf084fsrgznpw7-python3.6-jupyterhub-ldapauthenticator-1.1
strip is /nix/store/lvx1acn1ig1j2km8jds5x3ggh3f2wa8v-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/a95dawf4zpxfmy9qcdlf084fsrgznpw7-python3.6-jupyterhub-ldapauthenticator-1.1/lib
patching script interpreter paths in /nix/store/a95dawf4zpxfmy9qcdlf084fsrgznpw7-python3.6-jupyterhub-ldapauthenticator-1.1
checking for references to /build in /nix/store/a95dawf4zpxfmy9qcdlf084fsrgznpw7-python3.6-jupyterhub-ldapauthenticator-1.1...
/nix/store/a95dawf4zpxfmy9qcdlf084fsrgznpw7-python3.6-jupyterhub-ldapauthenticator-1.1

@ixxie
Copy link
Contributor Author

ixxie commented Feb 25, 2018

@dotlambda - I applied the requested changes. I didn't use the disabled = !isPy3k because the github page for the package says this could in principle work on python2 but simply wasn't tested.

@FRidh FRidh merged commit e6cc9a7 into NixOS:master Feb 26, 2018
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.

None yet

4 participants