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

busybox: 1.31.1 -> 1.32.0 #104433

Merged
merged 1 commit into from Jan 3, 2021
Merged

busybox: 1.31.1 -> 1.32.0 #104433

merged 1 commit into from Jan 3, 2021

Conversation

j4m3s-s
Copy link
Member

@j4m3s-s j4m3s-s commented Nov 20, 2020

Things done

The patches removed are on the 1.32.0 branch now.

Building works and nix-build pkgs/stdenv/linux/make-bootstrap-tools.nix -A test also works as asked in the file.

  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • 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 nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@j4m3s-s
Copy link
Member Author

j4m3s-s commented Nov 20, 2020

@TethysSvensson

@TethysSvensson
Copy link
Contributor

This looks good to me. It is a (hopefully simple) version bump with two patches removed.

What I've tested that:

  • It builds
  • A few of the applets work as expected
  • The removed patches are in fact present upstream
  • The fix for the udhcpc script still works as expected

@rissson
Copy link
Member

rissson commented Nov 24, 2020

Result of nixpkgs-review pr 104433 1

2 packages marked as broken and skipped:
  • hercules-ci-agent
  • nix-exec
2 packages failed to build:
  • fusionInventory
  • python37Packages.nixpkgs
60 packages built:
  • bundix
  • busybox
  • busybox-sandbox-shell
  • cabal2nix
  • cachix
  • common-updater-scripts
  • crate2nix
  • crystal2nix
  • dep2nix
  • disnix
  • disnixos
  • epson-escpr2
  • fcron
  • go2nix
  • hydra-unstable
  • lazpaint
  • lightdm
  • lightdm-enso-os-greeter
  • lightdm-mini-greeter
  • lightdm-tiny-greeter
  • lightdm_gtk_greeter
  • lightdm_qt
  • lispPackages.quicklisp-to-nix
  • lispPackages.quicklisp-to-nix-system-info
  • molly-guard
  • nix (nixStable)
  • nix-binary-cache
  • nix-bundle
  • nix-direnv
  • nix-doc
  • nix-du
  • nix-index
  • nix-pin
  • nix-plugins
  • nix-prefetch
  • nix-prefetch-bzr
  • nix-prefetch-cvs
  • nix-prefetch-docker
  • nix-prefetch-git
  • nix-prefetch-hg
  • nix-prefetch-scripts
  • nix-prefetch-svn
  • nix-review (nixpkgs-review)
  • nix-serve
  • nix-update
  • nix-update-source
  • nixFlakes (nixUnstable)
  • nixos-generators
  • nixos-shell
  • nixui
  • pantheon.elementary-greeter
  • pipework
  • python37Packages.nix-kernel
  • python37Packages.pythonix
  • python38Packages.nix-kernel
  • python38Packages.nixpkgs
  • python38Packages.pythonix
  • update-nix-fetchgit
  • vgo2nix
  • vulnix

@rissson
Copy link
Member

rissson commented Nov 24, 2020

It's weird python38Packages.nixpkgs builds but not python37Packages.nixpkgs...

@TethysSvensson
Copy link
Contributor

This is mostly replicable on my system:

Result of nixpkgs-review pr 104433 1

1 package marked as broken and skipped:
  • nix-exec
2 packages blacklisted:
  • tests.nixos-functions.nixos-test
  • tests.nixos-functions.nixosTest-test
3 packages failed to build:
  • fusionInventory
  • pantheon.elementary-greeter
  • python37Packages.nixpkgs
62 packages built:
  • bundix
  • busybox
  • busybox-sandbox-shell
  • cabal2nix
  • cachix
  • common-updater-scripts
  • crate2nix
  • crystal2nix
  • dep2nix
  • disnix
  • disnixos
  • epson-escpr2
  • fcron
  • go2nix
  • haskellPackages.cachix
  • haskellPackages.hocker
  • haskellPackages.nix-paths
  • haskellPackages.update-nix-fetchgit
  • hydra-unstable
  • lazpaint
  • lightdm
  • lightdm-enso-os-greeter
  • lightdm-mini-greeter
  • lightdm-tiny-greeter
  • lightdm_gtk_greeter
  • lightdm_qt
  • lispPackages.quicklisp-to-nix
  • lispPackages.quicklisp-to-nix-system-info
  • molly-guard
  • nix (nixStable)
  • nix-bundle
  • nix-direnv
  • nix-doc
  • nix-du
  • nix-index
  • nix-pin
  • nix-plugins
  • nix-prefetch
  • nix-prefetch-bzr
  • nix-prefetch-cvs
  • nix-prefetch-docker
  • nix-prefetch-git
  • nix-prefetch-hg
  • nix-prefetch-scripts
  • nix-prefetch-svn
  • nix-serve
  • nix-update
  • nix-update-source
  • nixFlakes (nixUnstable)
  • nixos-generators
  • nixos-shell
  • nixpkgs-review
  • nixui
  • pipework
  • python37Packages.nix-kernel
  • python37Packages.pythonix
  • python38Packages.nix-kernel
  • python38Packages.nixpkgs
  • python38Packages.pythonix
  • update-nix-fetchgit
  • vgo2nix
  • vulnix

However I as far as I can tell python37Packages.nixpkgs is broken on master as well.

@rissson
Copy link
Member

rissson commented Nov 24, 2020

It seems alright to merge though. @j4m3s-s do you have any time to look into why those few dependencies fail, if they weren't broken on master before?

Copy link
Contributor

@r-burns r-burns left a comment

Choose a reason for hiding this comment

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

python37Packages.nixpkgs is broken due to python37Packages.pythonix, which builds but fails to import on non-python-3.8 (see #103151 (comment)). I think this is ready to be merged.

Result of nixpkgs-review pr 104433 1

1 package marked as broken and skipped:
  • nix-exec
2 packages blacklisted:
  • tests.nixos-functions.nixos-test
  • tests.nixos-functions.nixosTest-test
1 package failed to build:
  • python37Packages.nixpkgs
64 packages built:
  • bundix
  • busybox
  • busybox-sandbox-shell
  • cabal2nix
  • cachix
  • common-updater-scripts
  • crate2nix
  • crystal2nix
  • dep2nix
  • disnix
  • disnixos
  • epson-escpr2
  • fcron
  • fusionInventory
  • go2nix
  • haskellPackages.cachix
  • haskellPackages.hocker
  • haskellPackages.nix-paths
  • haskellPackages.update-nix-fetchgit
  • hydra-unstable
  • lazpaint
  • lightdm
  • lightdm-enso-os-greeter
  • lightdm-mini-greeter
  • lightdm-tiny-greeter
  • lightdm_gtk_greeter
  • lightdm_qt
  • lispPackages.quicklisp-to-nix
  • lispPackages.quicklisp-to-nix-system-info
  • molly-guard
  • nix (nixStable)
  • nix-bundle
  • nix-direnv
  • nix-doc
  • nix-du
  • nix-index
  • nix-pin
  • nix-plugins
  • nix-prefetch
  • nix-prefetch-bzr
  • nix-prefetch-cvs
  • nix-prefetch-docker
  • nix-prefetch-git
  • nix-prefetch-hg
  • nix-prefetch-scripts
  • nix-prefetch-svn
  • nix-serve
  • nix-update
  • nix-update-source
  • nixFlakes (nixUnstable)
  • nixos-generators
  • nixos-shell
  • nixpkgs-review
  • nixui
  • pantheon.elementary-greeter
  • pipework
  • python37Packages.nix-kernel
  • python37Packages.pythonix
  • python38Packages.nix-kernel
  • python38Packages.nixpkgs
  • python38Packages.pythonix
  • update-nix-fetchgit
  • vgo2nix
  • vulnix

@SuperSandro2000
Copy link
Member

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.

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

1 package marked as broken and skipped:
  • nix-exec
2 packages blacklisted:
  • tests.nixos-functions.nixos-test
  • tests.nixos-functions.nixosTest-test
3 packages failed to build and are new build failures:
62 packages built:
  • bundix
  • busybox
  • busybox-sandbox-shell
  • cabal2nix
  • common-updater-scripts
  • crate2nix
  • crystal2nix
  • dep2nix
  • disnix
  • disnixos
  • epson-escpr2
  • fcron
  • fusionInventory
  • go2nix
  • haskellPackages.hocker
  • haskellPackages.nix-paths
  • haskellPackages.update-nix-fetchgit
  • hydra-unstable
  • lazpaint
  • lightdm
  • lightdm-enso-os-greeter
  • lightdm-mini-greeter
  • lightdm-tiny-greeter
  • lightdm_gtk_greeter
  • lightdm_qt
  • lispPackages.quicklisp-to-nix
  • lispPackages.quicklisp-to-nix-system-info
  • molly-guard
  • nix (nixStable)
  • nix-bundle
  • nix-direnv
  • nix-doc
  • nix-du
  • nix-index
  • nix-pin
  • nix-plugins
  • nix-prefetch
  • nix-prefetch-bzr
  • nix-prefetch-cvs
  • nix-prefetch-docker
  • nix-prefetch-git
  • nix-prefetch-hg
  • nix-prefetch-scripts
  • nix-prefetch-svn
  • nix-serve
  • nix-update
  • nix-update-source
  • nixFlakes (nixUnstable)
  • nixos-generators
  • nixos-shell
  • nixpkgs-review
  • nixui
  • pantheon.elementary-greeter
  • pipework
  • python37Packages.nix-kernel
  • python37Packages.pythonix
  • python38Packages.nix-kernel
  • python38Packages.nixpkgs
  • python38Packages.pythonix
  • update-nix-fetchgit
  • vgo2nix
  • vulnix

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

python38Packages.nixpkgs:

Zero tests run by pytest got detected: 'Ran 0 tests in 0.000s'
You have the following options to fix this:

  • Tell pytest(CheckHook) where to find the tests included in the package
  • Check if the GitHub Repo contains tests but they are not shipped with Pypi. If so please switch to fetchFromGitHub.
  • If the Packages does not contain any tests add 'doCheck = false;'and apythonImportsCheck`.

@SuperSandro2000
Copy link
Member

All build failures are known and unrelated to this PR.

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

6 participants