Allow symbolic links

[offlinehacker]

This is worng: https://github.com/kevva/bin-wrapper/blob/master/index.js#L213

You should allow symbolic links, there's no reason why not, and that's exactly the use case we have in nix package manager and in nixos.

[kevva]

Ok, so what did you mean by #16?

[offlinehacker]

I will do some more testing and report you, but currently without #16 it
seems to work fine or at least it does not break and picks local binaries
:) It might be that the problem was in other library.

It might still be good to symlink global binaries to vendor folder. Then,
when you would run binary using bin-wrapper it would prefer binaries
specified through PATH, or else it would use symlinked ones. This way you
get deterministic environment and i think you would handle pretty much all
use cases.
On Mar 26, 2014 4:21 PM, "Kevin Mårtensson" notifications@github.com
wrote:

Ok, so what did you meant by #16#16
?

Reply to this email directly or view it on GitHubhttps://github.com//issues/18#issuecomment-38696822
.

[offlinehacker]

After you fix https://github.com/kevva/find-file/issues/1 it should be fine. You should also remove 5b84720 or symlinks don't work.