Added a new option evn_rvm #77

Merged
merged 2 commits into from Mar 16, 2013

Projects

None yet

3 participants

@tanaka51
Contributor

Added a new option env_rvm.

I commonly type rrvm env . -- --env > .powenv to specify rvm gemset path for Pow.
See also http://stackoverflow.com/questions/10154928/pow-rvm-and-zsh-not-working-together

Thank you.

@rodreegez rodreegez and 1 other commented on an outdated diff Mar 13, 2013
@@ -348,6 +348,16 @@ module Powder
end
+ desc "env_rvm", "Create or add rvm env to .powenv"
+ def env_rvm
+ if File.exists?(".rvmrc")
+ say %x{rvm env . -- --env >> .powenv}
+ show_env
+ else
+ say ".rvmrc is not exists."
rodreegez
rodreegez Mar 13, 2013 Collaborator

Nitpicking over grammer - this should read ".rvmrc does not exist."

tanaka51
tanaka51 Mar 13, 2013 Contributor

Thank you very much!

rodreegez
rodreegez Mar 13, 2013 Collaborator

No problem. Thanks so much for helping out!

Contributor

@rodreegez Thank you so much for advice! I fixed grammar.

@csiszarattila csiszarattila merged commit 1a7995c into powder-rb:master Mar 16, 2013
@csiszarattila csiszarattila added a commit that referenced this pull request Mar 16, 2013
@csiszarattila csiszarattila Addition to pull request: #77: use the generic POW_ENV const for refe…
…ncing to the .powenv file
96bc371
Collaborator

Thanks for the contribution!

I merged and hope you dont mind but I made some small changes just to avoid possible file access errors.

Contributor

@csiszarattila Thank you for merge this PR and fix some issues!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment