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

systemd: fix 0019-*.patch #101791

Merged
merged 1 commit into from Oct 27, 2020
Merged

systemd: fix 0019-*.patch #101791

merged 1 commit into from Oct 27, 2020

Conversation

flokli
Copy link
Contributor

@flokli flokli commented Oct 26, 2020

This should be zero functionality change, just make tooling better.
I'm currently preparing a local 247-rc1 and realized the patches
don't apply properly.

This was introduced in #99621, but
didn't follow the process documented in
pkgs/os-specific/linux/systemd/default.nix, namely, the git am and
git format-patch workflow, which caused
0019-revert-get-rid-of-seat_can_multi_session.patch to not apply with
git am due to missing authorship information.

I did apply this patch manually, and copied authorship information from
4e384dd.

Motivation for this change

Make these lines be accurate again:

# If these need to be regenerated, `git am path/to/00*.patch` them into a
# systemd worktree, rebase to the more recent systemd version, and export the
# patches again via `git format-patch v${version}`.
Things done
  • 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.

This was introduced in NixOS#99621, but
didn't follow the process documented in
pkgs/os-specific/linux/systemd/default.nix, namely, the `git am` and
`git format-patch` workflow, which caused
`0019-revert-get-rid-of-seat_can_multi_session.patch` to not apply with
`git am` due to missing authorship information.

I did apply this patch manually, and copied authorship information from
4e384dd.
@picnoir picnoir merged commit 5e2fde0 into NixOS:staging Oct 27, 2020
systemd automation moved this from To Do to Done Oct 27, 2020
@flokli flokli deleted the systemd-fix-patch branch October 27, 2020 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
systemd
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants