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: Add a default udhcpc dispatcher script #92581
Conversation
The udhcpc binary which currently ships as part of the busybox derivation will by default search for a dispatcher script at the location /usr/share/udhcpc/default.script. This commit includes a working default script with udhcpc and updates the location where udhcpc searches for this script. The script was taken the script from the udhcpc package in debian buster. The only changes from that script is to make it use paths from the nix store and remove the run-time check for /sbin/resolvconf.
/marvin opt-in |
Hi! I'm an experimental bot. My goal is to guide this PR through its stages, hopefully ending with a merge. You can read up on the usage here. |
/status needs_reviewer |
I noticed that this package does not have a maintainer. I would be okay with being added as a maintainer, though I am not yet a nix maintainer. Just to be sure, is that anything else to do than adding myself to |
@matthewbauer Would you be able to review this PR or help me find someone who can? |
@grahamc I do not understand what this CI error means. What does |
Result of 2 packages blacklisted:- tests.nixos-functions.nixos-test - tests.nixos-functions.nixosTest-test 7 packages failed to build:- deepin.dde-kwin - deepin.dde-launcher - deepin.dde-session-ui - deepin.startdde - hydra-unstable - nix-exec - python37Packages.nixpkgs 60 packages built:- bundix - busybox - busybox-sandbox-shell - cabal2nix - cachix - common-updater-scripts - crystal2nix - dep2nix - disnix - disnixos - epson-escpr2 - fcron - fusionInventory - go2nix - haskellPackages.cachix - haskellPackages.hercules-ci-agent - haskellPackages.nix-paths - hercules-ci-agent - hydra-migration - 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-build-uncached - nix-bundle - nix-direnv - 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 - pantheon.nixpkgs-github-update - pipework - python37Packages.pythonix - python38Packages.nixpkgs - python38Packages.pythonix - vgo2nix - vulnix As far as I can tell, those 7 packages are also broken on master. |
Motivation for this change
The udhcpc binary which currently ships as part of the busybox
derivation will by default search for a dispatcher script at the
location /usr/share/udhcpc/default.script.
It would be much nicer if udhcpc had a functional default script.
Things done
This commit includes a working default script with udhcpc and updates
the location where udhcpc searches for this script.
The script was taken the script from the udhcpc package in debian
buster. The only changes from that script is to make it use paths from
the nix store and remove the run-time check for /sbin/resolvconf.
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)