Inconsistency between `which` and `exec` #21

Closed
miyagawa opened this Issue Apr 7, 2013 · 2 comments

Comments

Projects
None yet
2 participants
@miyagawa
Collaborator

miyagawa commented Apr 7, 2013

Not a big deal, but which seems to locate commands outside perl's bin, while exec fails to. It looks like an inconsistent behavior - compare with rbenv.

➜  ~  rbenv which ls
rbenv: ls: command not found
➜  ~  rbenv exec ls
rbenv: ls: command not found
➜  ~  plenv which ls
/bin/ls
➜  ~  plenv exec ls
[plenv] There is no ls in /Users/miyagawa/.plenv/versions/5.16.3/bin.(determined by /Users/miyagawa/.plenv/version)
@tokuhirom

This comment has been minimized.

Show comment Hide comment
@tokuhirom

tokuhirom Apr 8, 2013

Owner

Oops. It's a bug.

Owner

tokuhirom commented Apr 8, 2013

Oops. It's a bug.

tokuhirom added a commit that referenced this issue Apr 24, 2013

Merge pull request #24 from syohex/fix-which
Search only plenv PATH if current environment is not "system"(#21)
@tokuhirom

This comment has been minimized.

Show comment Hide comment
@tokuhirom

tokuhirom Apr 24, 2013

Owner

Fixed by syohex++

Owner

tokuhirom commented Apr 24, 2013

Fixed by syohex++

@tokuhirom tokuhirom closed this Apr 24, 2013

tokuhirom added a commit that referenced this issue Apr 25, 2013

Checking in changes prior to tagging of version 1.4.13.
Changelog diff is:

diff --git a/Changes b/Changes
index 4e3224a..0dd2031 100644
--- a/Changes
+++ b/Changes
@@ -2,6 +2,13 @@ Revision history for Perl extension App::plenv

 {{$NEXT}}

+1.4.13 2013-04-24T05:33:09Z
+
+    - Do not handle `plenv exec perl --version` in core.
+      (tokuhirom)
+    - Search only plenv PATH if current environment is not "system"(#21)
+      (syohex)
+
 1.4.12 2013-04-15T05:43:50Z

     - Added `plenv --version`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment