Skip to content

Retire systemdMinimal#264163

Closed
blitz wants to merge 15 commits intoNixOS:stagingfrom
blitz:systemd-libs
Closed

Retire systemdMinimal#264163
blitz wants to merge 15 commits intoNixOS:stagingfrom
blitz:systemd-libs

Conversation

@blitz
Copy link
Copy Markdown
Contributor

@blitz blitz commented Oct 29, 2023

Description of changes

This PR retires systemdMinimal. It's a follow-up to #261798. The goal is to avoid having multiple sets of systemd binaries in a typical system closure and save everyone some disk space. This is especially relevant for smaller image-based systems (see #263462).

We replace systemdMinimal with:

  • systemd, if systemd commands or udevadm is needed (bluez) or when systemd is already in the closure (dbus)
  • systemdLibs
    • for dependencies of systemd itself to break dependencies
    • for packages that just need libudev or libsystemd that don't already have systemd in their closure.

Some of the uses of systemdLibs could just be systemd, if we manage to give pkgs.systemd a lib output that doesn't drag in lots of unneeded binaries.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • 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/)
  • 23.11 Release Notes (or backporting 23.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
  • Fits CONTRIBUTING.md.

@github-actions github-actions bot added 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: module (update) This PR changes an existing module in `nixos/` 6.topic: systemd Software suite that provides an array of system components for Linux operating systems. labels Oct 29, 2023
@blitz blitz self-assigned this Oct 30, 2023
@wegank wegank added the 2.status: merge conflict This PR has merge conflicts with the target branch label Mar 20, 2024
@blitz
Copy link
Copy Markdown
Contributor Author

blitz commented May 22, 2024

I unfortunately have no time to finish this. 👎

@blitz blitz closed this May 22, 2024
@rhendric rhendric mentioned this pull request Feb 14, 2026
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2.status: merge conflict This PR has merge conflicts with the target branch 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 6.topic: systemd Software suite that provides an array of system components for Linux operating systems. 8.has: module (update) This PR changes an existing module in `nixos/`

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants