Skip to content

Improve path support#60

Merged
pichillilorenzo merged 1 commit intopichillilorenzo:masterfrom
dnicolson:improve-path-support
Jun 10, 2018
Merged

Improve path support#60
pichillilorenzo merged 1 commit intopichillilorenzo:masterfrom
dnicolson:improve-path-support

Conversation

@dnicolson
Copy link
Copy Markdown
Contributor

@dnicolson dnicolson commented Jun 10, 2018

This pull request removes some code from the SublimeFixMacPath project in favour of using shellenv which is a Package Control dependency.

Currently when using the fish shell this package can break others that depend on the PATH environment variable. Initially a change was made (#59) to support the fish shell but it did not work as expected. In the original project fish support was added (int3h/SublimeFixMacPath#7) but broken soon after (int3h/SublimeFixMacPath@fecd771).

It is very likely more code can be removed by using shellenv, but there could be some edge cases @pichillilorenzo?

@pichillilorenzo
Copy link
Copy Markdown
Owner

Thanks @dnicolson! However I don't know if there could be some edge cases!

@pichillilorenzo pichillilorenzo merged commit e37c743 into pichillilorenzo:master Jun 10, 2018
@dnicolson dnicolson deleted the improve-path-support branch June 10, 2018 23:36
@pichillilorenzo
Copy link
Copy Markdown
Owner

Hey @dnicolson, I want ask you if you want help me develop this project! I'm searching for members that want help me in their spare time! See #51 for details 😄. Thanks anyway!

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.

2 participants