overlay hunt: Remove systemd overlay: upstream patch merged#1843
Merged
brianmcgillion merged 2 commits intotiiuae:mainfrom Mar 22, 2026
Merged
overlay hunt: Remove systemd overlay: upstream patch merged#1843brianmcgillion merged 2 commits intotiiuae:mainfrom
brianmcgillion merged 2 commits intotiiuae:mainfrom
Conversation
brianmcgillion
approved these changes
Mar 21, 2026
vunnyso
approved these changes
Mar 22, 2026
Collaborator
vunnyso
left a comment
There was a problem hiding this comment.
If the pre-merge failures are unrelated to this PR, then LGTM.
The pam_systemd_home PAM_TEXT_INFO patch (systemd/systemd#40412) has been merged upstream and is now available in the locked nixpkgs revision. This overlay was causing a massive rebuild cascade: since systemd is a transitive dependency of virtually every package (via dbus, polkit, udev, glib, gtk, qt, etc.), patching it forced from-source rebuilds of packages like inkscape, openjdk, qtbase6, and hundreds of others that would otherwise be fetched from the binary cache. Signed-off-by: vadik likholetov <vadikas@gmail.com>
8ea0539 to
6a445b3
Compare
The existing cross-compilation overlay fixed EFI_ARCH detection and AR/RANLIB but missed three issues causing build failures: sbsigntool: - CCAN configurator compiled for target arch could not execute on build host. Fix: override CC to native compiler for create-ccan-tree. - ./configure lacked --host/--build flags for cross-compilation. - help2man cannot run cross-compiled binaries. Fix: exclude docs. efitools: - Default `all` target runs freshly-built tools (cert-to-efi-sig-list, sign-efi-sig-list) for cert generation, which fails when cross-compiled. Fix: build only CLI binaries, skip cert/auth generation. Signed-off-by: vadik likholetov <vadikas@gmail.com>
468514d to
458203f
Compare
brianmcgillion
approved these changes
Mar 22, 2026
19 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
pam_systemd_homePAM_TEXT_INFO overlay for systemd, as the upstream patch (systemd/systemd#40412) has been merged and is available in the current locked nixpkgs revision (b40629efe5d6)Test plan
nix build .#lenovo-x1-carbon-gen11-debug --dry-runshows significantly fewer packages to build from source