diff options
| author | Josh Heinrichs <josh.heinrichs@shopify.com> | 2024-10-04 17:10:06 -0600 |
|---|---|---|
| committer | Josh Heinrichs <josh.heinrichs@shopify.com> | 2025-04-08 13:47:06 -0600 |
| commit | 74e7f9768b19ae1a3b75632db42b25bf1011ac5e (patch) | |
| tree | c9ce539e886793f1f5d19e5b57c3f938b958eb51 /pkgs/development/python-modules/python-mapnik/python-mapnik_std_optional.patch | |
| parent | b89da0c38097524297207a993dcb2fe9de9e3eb8 (diff) | |
git: move credential helpers to exec path
Currently if the credential helpers packaged with git aren't on your
PATH, git will fail to locate them. This most notably affects git
maintanence[1] which generates systemd units / launchd agents that
invoke git directly without setting PATH. Home Manager is also able to
generate maintanence units for systemd and does not set PATH[2], meaning
credential helpers will not work without additional configuration.
Moving the credential helpers to git's exec path[3] allows git to locate
them regardless of how PATH is set, and matches the packaging of
Fedora[4] and Homebrew[5]. Typically credential helpers are not added to
PATH unless they are provided by another package separate from git (e.g.
git-credential-gcloud).
I opted to create symlinks at the old bin paths in case some users were
relying on invoking them directly. In practice I doubt this is done
much. If we want to slim down git's bin dir I think the symlinks could
be removed, and in doing so we'd better match the packaging done in
other distributions.
[1]: https://git-scm.com/docs/git-maintenance/2.47.1
[2]: https://github.com/nix-community/home-manager/blob/b6fd653ef8fbeccfd4958650757e91767a65506d/modules/programs/git.nix#L619
[3]: https://git-scm.com/docs/git/2.47.1#Documentation/git.txt---exec-pathltpathgt
[4]: https://src.fedoraproject.org/rpms/git/blob/rawhide/f/git.spec#_651
[5]: https://github.com/Homebrew/homebrew-core/blob/12120ff79ce009489b958d8271af686d74403859/Formula/g/git.rb#L112
Diffstat (limited to 'pkgs/development/python-modules/python-mapnik/python-mapnik_std_optional.patch')
0 files changed, 0 insertions, 0 deletions
