rvm shouldn't set MANPATH #966

Merged
merged 1 commit into from May 17, 2012

Conversation

Projects
None yet
3 participants
Contributor

thirtyseven commented May 16, 2012

RVM setting MANPATH will break the system manpath and isn't necessary for looking up manpages for commands in .rvm/bin.

This reverts commit 1f68d64.

This pull request passes (merged 1e41be0 into ecb2139).

Contributor

thirtyseven commented May 16, 2012

A test for the correct behavior:

man -P true man # status=0;
man -P true rvm # status=0;

@mpapis mpapis added a commit that referenced this pull request May 17, 2012

@mpapis mpapis Merge pull request #966 from thirtyseven/master
rvm shouldn't set MANPATH
50995d7

@mpapis mpapis merged commit 50995d7 into rvm:master May 17, 2012

mpapis was assigned May 17, 2012

Owner

mpapis commented May 17, 2012

just mentioning #880

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