Output correct environment path. #3483
Conversation
|
The build is failing here:
i noticed that the way the command is being tested is not as the one explained in the documentation of rvm. https://rvm.io/deployment/cron#loading-rvm-environment-files-in-shell-script to fix this i need to know what is the correct way to execute the command, thanks. |
scripts/functions/env
Outdated
| @@ -272,7 +272,26 @@ __rvm_env_print() | |||
| if | |||
| [[ "$rvm_path_flag" == "1" || "$*" == *"--path"* ]] | |||
| then | |||
| echo "$environment_file_path" | |||
| if | |||
| [[ ! -z "$@" ]] | |||
There was a problem hiding this comment.
-n is a tad more concise than ! -z for the string-isn't-null test.
There was a problem hiding this comment.
you are right @havenwood thanks, i will update it.
| @@ -656,6 +656,7 @@ __rvm_parse_args() | |||
| rvm_token=${rvm_token#--} | |||
| rvm_token=${rvm_token//-/_} | |||
| export "rvm_${rvm_token}_flag"=1 | |||
| rvm_ruby_args+=($next_token); | |||
There was a problem hiding this comment.
this does not look good, the options here do not take arguments and next_token is what's listed after them, could be anything
Pull requets to fix the "'env' command doesn't operate as expected #3465" issue