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

python3Packages.nbclient: 0.5.8 -> 0.5.9, fix build #146813

Merged
merged 1 commit into from
Nov 24, 2021

Conversation

erictapen
Copy link
Member

This also fixes the build failure introduced in #146151, as for some reason the bin directory is empty when doCheck = true and then the postFixup failed.

0.5.9 contains jupyter/nbclient#173 so no need for the postFixup anymore.

This should also close #146277.
The original issue was #145968.

Run

nix build --impure --expr "let pkgs = import ./. {}; in pkgs.python3.withPackages (ps: [ ps.jupyterlab ])"

for testing.

Motivation for this change
Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 21.11 Release Notes (or backporting 21.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

This also unbreaks the build, as for some reason the bin directory is
empty when doCheck = true and then the postFixup fails.
@SuperSandro2000
Copy link
Member

@cole-h
@ofborg eval

@ofborg ofborg bot added 11.by: package-maintainer This PR was created by the maintainer of the package it changes 10.rebuild-darwin: 101-500 10.rebuild-linux: 101-500 labels Nov 22, 2021
@SuperSandro2000
Copy link
Member

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).

Result of nixpkgs-review pr 146813 run on x86_64-linux 1

4 packages marked as broken and skipped:
  • cq-editor
  • python38Packages.skein
  • python38Packages.spyder
  • python39Packages.skein
30 packages failed to build and already failed to build on hydra master:
  • python38Packages.atomman: log was empty
  • python38Packages.ipydatawidgets: log was empty
  • python38Packages.ipympl: log was empty
  • python38Packages.potentials: log was empty
  • python38Packages.qiskit: log was empty
  • python38Packages.qiskit-aer: log was empty
  • python38Packages.qiskit-aqua: log was empty
  • python38Packages.qiskit-finance: log was empty
  • python38Packages.qiskit-ibmq-provider: log was empty
  • python38Packages.qiskit-ignis: log was empty
  • python38Packages.qiskit-machine-learning: log was empty
  • python38Packages.qiskit-nature: log was empty
  • python38Packages.qiskit-optimization: log was empty
  • python38Packages.qiskit-terra: log was empty
  • python38Packages.robotframework-tools: log was empty
  • python39Packages.atomman: log was empty
  • python39Packages.ipydatawidgets: log was empty
  • python39Packages.ipympl: log was empty
  • python39Packages.potentials: log was empty
  • python39Packages.qiskit: log was empty
  • python39Packages.qiskit-aer: log was empty
  • python39Packages.qiskit-aqua: log was empty
  • python39Packages.qiskit-finance: log was empty
  • python39Packages.qiskit-ibmq-provider: log was empty
  • python39Packages.qiskit-ignis: log was empty
  • python39Packages.qiskit-machine-learning: log was empty
  • python39Packages.qiskit-nature: log was empty
  • python39Packages.qiskit-optimization: log was empty
  • python39Packages.qiskit-terra: log was empty
  • python39Packages.robotframework-tools: log was empty
1 package failed to build and are new build failure:
  • python38Packages.datashader: log was empty
102 packages built:
  • ihaskell
  • jupyter
  • opsdroid
  • python38Packages.Nikola
  • python38Packages.Pweave
  • python38Packages.ansible-kernel
  • python38Packages.batchspawner
  • python38Packages.cufflinks
  • python38Packages.dockerspawner
  • python38Packages.einops
  • python38Packages.holoviews
  • python38Packages.hvplot
  • python38Packages.intake
  • python38Packages.ipyvue
  • python38Packages.ipyvuetify
  • python38Packages.ipywidgets
  • python38Packages.jupyter
  • python38Packages.jupyter-server-mathjax
  • python38Packages.jupyter-sphinx
  • python38Packages.jupyter_server
  • python38Packages.jupyterhub
  • python38Packages.jupyterhub-ldapauthenticator
  • python38Packages.jupyterhub-systemdspawner
  • python38Packages.jupyterhub-tmpauthenticator
  • python38Packages.jupyterlab
  • python38Packages.jupyterlab-git
  • python38Packages.jupyterlab_launcher
  • python38Packages.jupyterlab_server
  • python38Packages.jupytext
  • python38Packages.kmapper
  • python38Packages.lightparam
  • python38Packages.livelossplot
  • python38Packages.modeled
  • python38Packages.moretools
  • python38Packages.nbclassic
  • python38Packages.nbclient
  • python38Packages.nbconflux
  • python38Packages.nbconvert
  • python38Packages.nbdime
  • python38Packages.nbsmoke
  • python38Packages.nbsphinx
  • python38Packages.nix-kernel
  • python38Packages.notebook
  • python38Packages.notedown
  • python38Packages.oauthenticator
  • python38Packages.papermill
  • python38Packages.phik
  • python38Packages.pydeck
  • python38Packages.stytra
  • python38Packages.vega
  • python38Packages.widgetsnbextension
  • python38Packages.zetup
  • python39Packages.Nikola
  • python39Packages.Pweave
  • python39Packages.ansible-kernel
  • python39Packages.batchspawner
  • python39Packages.cufflinks
  • python39Packages.datashader
  • python39Packages.dockerspawner
  • python39Packages.einops
  • python39Packages.holoviews
  • python39Packages.hvplot
  • python39Packages.intake
  • python39Packages.ipyvue
  • python39Packages.ipyvuetify
  • python39Packages.ipywidgets
  • python39Packages.jupyter
  • python39Packages.jupyter-server-mathjax
  • python39Packages.jupyter-sphinx
  • python39Packages.jupyter_server
  • python39Packages.jupyterhub
  • python39Packages.jupyterhub-ldapauthenticator
  • python39Packages.jupyterhub-systemdspawner
  • python39Packages.jupyterhub-tmpauthenticator
  • python39Packages.jupyterlab
  • python39Packages.jupyterlab-git
  • python39Packages.jupyterlab_launcher
  • python39Packages.jupyterlab_server
  • python39Packages.jupytext
  • python39Packages.kmapper
  • python39Packages.lightparam
  • python39Packages.livelossplot
  • python39Packages.modeled
  • python39Packages.moretools
  • python39Packages.nbclassic
  • python39Packages.nbclient
  • python39Packages.nbconflux
  • python39Packages.nbconvert
  • python39Packages.nbdime
  • python39Packages.nbsmoke
  • python39Packages.nbsphinx
  • python39Packages.nix-kernel
  • python39Packages.notebook
  • python39Packages.notedown
  • python39Packages.oauthenticator
  • python39Packages.papermill
  • python39Packages.phik
  • python39Packages.pydeck
  • python39Packages.stytra
  • python39Packages.vega
  • python39Packages.widgetsnbextension
  • python39Packages.zetup

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:

ihaskell:

warning: unused-argument
Unused argument: paths.
Near pkgs/build-support/trivial-builders.nix:372:12:

    |
372 |          , paths
    |            ^

@SuperSandro2000 SuperSandro2000 merged commit c669e5f into NixOS:master Nov 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Jupyterlab fails to install, giving a collision error
2 participants