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: add share dir to pathsToLink #21957
Conversation
@kjanosz, thanks for your PR! By analyzing the history of the files in this pull request, we identified @scolobb, @NeQuissimus and @grahamc to be potential reviewers. |
@@ -15,6 +15,8 @@ stdenv.mkDerivation rec { | |||
sha256 = "1adgj0p4c8aq9rpkv33k8ra69922vfkjw63b666i66v6zr0s8znp"; | |||
}; | |||
|
|||
environment.pathsToLink = [ "/share/oh-my-zsh" ]; |
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.
What does this give us?
I haven't found anything looking like a doc for this attribute :-(
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.
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.
Oh, nice search engine, didn't know it existed, thanks a lot!
Your change seems reasonable to me.
@@ -15,6 +15,8 @@ stdenv.mkDerivation rec { | |||
sha256 = "1adgj0p4c8aq9rpkv33k8ra69922vfkjw63b666i66v6zr0s8znp"; | |||
}; | |||
|
|||
environment.pathsToLink = [ "/share/oh-my-zsh" ]; |
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.
Note that I removed the |
My apologies then - I have this line in private module config and didn't actually fully check that in package |
No worries, that's why I went to build it myself. |
Motivation for this change
Some window/desktop managers do not add
share
directory or do it selectively.Without it
oh-my-zsh
is not linked to/run/current-system/sw/share
, which then makes it harder to set$ZSH
variable in.zshrc
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)