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
oh-my-zsh: cleanups, don't require perl in pygmalion theme anymore #96532
Conversation
- use fetchFromGitHub instead of fetchgit - indent the `installPhase` properly - remove unused `pathsToLink` - don't hardcode `phases` to allow patching - don't copy from $src, but . to allow patching
@flokli Will this patch work with BSD/macOS |
There was another (preferrable IMHO) approach proposed in ohmyzsh/ohmyzsh#9210 (comment), which doesn't need to shell out to anything anymore. |
85a292d
to
cba993a
Compare
I pushed updated this PR with the proposed solution, if people want to tinker with it. @NeQuissimus I'll leave it on you to decide on whether to merge this before it's merged upstream, or wait what happens there. |
Thank you @flokli for the PR. I don't have any particular issues with it. Upstream for Oh My Zsh seems to move quite quickly and this Nix package is automatically updated, so I'd say that applying the patch won't be necessary very soon. Maybe add a TODO comment reminding to remove the patch application once your change is merged upstream. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a TODO comment reminding to remove the patch application once the change is merged upstream.
NixOS#91213 removed `perl` from $PATH. This adds a patch to oh-my-zsh, using `sed` instead of `perl` to do the regexp substitution.
cba993a
to
b07a1dc
Compare
I updated the comment linking to the PR to become a TODO once this is merged. |
Motivation for this change
After switching to latest nixpkgs unstable, I ran into some error messages in my prompt:
This was due to #91213 removing
perl
from$PATH
.There's no reason
oh-my-zsh
needs to use perl for the regexp substitution, so I opened ohmyzsh/ohmyzsh#9210 upstream, and fetch the patch here.I did some cleanup in the
oh-my-zsh
derivation before, so patching is actually possible :-)Things done
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)