Skip to content

Conversation

@geoffharcourt
Copy link
Collaborator

OS X 10.11 changes the file where path_helper is invoked from
/etc/zshenv to /etc/zprofile. The change results in path_helper
rearranging the path in such a way that interferes with path changes
made in .zshenv.

This README update instructs users to rename /etc/zprofile to
/etc/zshenv so that .zshenv (and .zshenv.local) will be loaded
after path_helper is called, maintaining the desired precedence of
paths.

Thanks to @reshleman and @ventsislaf for investigating and offering
solutions.

@geoffharcourt geoffharcourt force-pushed the gh-fix-el-capitan-path branch 3 times, most recently from 8aaa41e to 9a3cc3d Compare October 2, 2015 18:44
OS X 10.11 changes the file where `path_helper` is invoked from
`/etc/zshenv` to `/etc/zprofile`. The change results in `path_helper`
rearranging the path in such a way that interferes with path changes
made in `.zshenv`.

This README update instructs users to rename `/etc/zprofile` to
`/etc/zshenv` so that `.zshenv` (and `.zshenv.local`) will be loaded
*after* `path_helper` is called, maintaining the desired precedence of
paths.

Thanks to @reshleman and @ventsislaf for investigating and offering
solutions.
@geoffharcourt geoffharcourt force-pushed the gh-fix-el-capitan-path branch from 9a3cc3d to 7555fc7 Compare October 2, 2015 18:44
@derekprior
Copy link
Contributor

looks good to me.

@geoffharcourt
Copy link
Collaborator Author

Thanks for the input and the review.

@geoffharcourt
Copy link
Collaborator Author

Merged as 15de0bc

@zillou
Copy link

zillou commented Oct 3, 2015

the command should be

% sudo mv /etc/{zprofile,zshenv}

no space after the comma.

@zillou
Copy link

zillou commented Oct 3, 2015

@geoffharcourt Raised a PR to fix this: #424

@geoffharcourt geoffharcourt deleted the gh-fix-el-capitan-path branch October 7, 2015 02:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants