-
-
Notifications
You must be signed in to change notification settings - Fork 230
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
Add environment variable #320
Conversation
You may want to describe your change in the readme as well: https://github.com/zplug/zplug#4-environment-variables-for-zplug |
What does this change bring? Why is it necessary? |
@@ -95,6 +95,7 @@ __zplug::core::core::prepare() | |||
# Add to the PATH | |||
path=( | |||
${ZPLUG_ROOT:+"$ZPLUG_ROOT/bin"} | |||
${ZPLUG_HOME:+"$ZPLUG_BIN"} |
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.
Whether merged or not, the code here seems to be wrong.
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.
Okay, I will fix it.
@@ -362,6 +362,10 @@ source ~/.zshrc_secret | |||
zplug "some/command", hook-build:"make && sudo make install" | |||
``` | |||
|
|||
#### `ZPLUG_BIN` | |||
|
|||
Defaults to `$ZPLUG_BIN`. You can change the save destination of the command's symbol link, eg `~/bin`. |
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.
s/symbol/symbolic/
s/eg/e.g./
@@ -362,6 +362,10 @@ source ~/.zshrc_secret | |||
zplug "some/command", hook-build:"make && sudo make install" | |||
``` | |||
|
|||
#### `ZPLUG_BIN` | |||
|
|||
デフォルトは `$ZPLUG_HOME/bin`。コマンドのシンボルリンクの保存先を変更することができる。例えば `~/bin` とか。 |
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.
s/シンボル/シンボリック/
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.
Please rebase your change. Sorry for the extra work.
I already did rebase, but did you have any inadequacies? |
Looks like he wants one commit from you ("add $ZPLUG_BIN variable"), not 4. |
Apparently I seem to have misunderstood. @Defman21: Thank you |
@@ -356,6 +356,10 @@ source ~/.zshrc_secret | |||
zplug "some/command", hook-build:"make && sudo make install" | |||
``` | |||
|
|||
#### `ZPLUG_BIN` | |||
|
|||
Defaults to `$ZPLUG_BIN`. You can change the save destination of the command's symbolic link, e.g. `~/bin`. |
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.
You mean: Defaults to $ZPLUG_HOME/bin
Add
$ZPLUG_BIN
.