Path fix #2

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@complynx

If not for this, on my machine with MinGW in %PATH% this won't work

@complynx complynx Path fix
If not for this, on my machine with MinGW in %PATH% this won't work
89d24cb
@grompe
Owner
grompe commented May 28, 2016

I don't understand what are you trying to fix here.

What doesn't work?

What's in your PATH?

@complynx
complynx commented May 28, 2016 edited

"find" is also part of MinGW (as ★NIX utility), and has slightly different API.
And MinGW path may appear before system. Or Cygwin, or git, or any other MinGW/Cygwin/★NIX based installation.

@complynx
complynx commented May 28, 2016 edited

`Microsoft Windows [Version 10.0.10586]
(c) Корпорация Майкрософт (Microsoft Corporation), 2015. Все права защищены.

C:\Users\complynx>where find
d:\MinGW\msys\1.0\bin\find.exe
C:\Windows\System32\find.exe

C:\Users\complynx>`

@grompe
Owner
grompe commented May 28, 2016

Is that MinGW default behavior?

Because I'm inclined to say that if you're overriding system32 programs, you're going to have a problem with more .bat files than just mine.

Anyway, I'll put a workaround for this, thanks!

@grompe grompe closed this in 3478b1b May 28, 2016
@complynx

I don't remember exactly, which of the mentioned installations has these modifiers, yet certainly there are some which have a checkbox for that. Maybe I should've just moved these paths to the back of my PATH. It will be Ok, if MinGW prepends it's own paths on startup correctly. Otherwise I'd rather leave it as is, because I use nix commands and env far more often, than pure cmd. Yet now I know that and would be more careful with bat's next time.
Also to say, there is a move in Windows dev team to provide smooth execution of unix code on win, based on cygwin environment, and it might appear in the path as well, before SYSTEM (though I can't test that, I'm not assigned to MS preview builds)

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