-
Notifications
You must be signed in to change notification settings - Fork 822
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix maven daemon detection on windows platform #3767
Conversation
String mavenDaemonSuffixDetection = Utilities.isWindows() ? ".cmd" : ""; | ||
String ex = Files.exists(Paths.get(mavenHome.getPath(), "bin", "mvnd" + mavenDaemonSuffixDetection)) ? "mvnd" : "mvn"; //NOI18N |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think we should probe here for mvnd.exe
instead of .cmd
because mvnd
is a binary. It is not super important but it would make it symmetrical.
Thanks for fixing this for windows btw. Feel free to force push.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change done :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for the fix. looks good.
Looks also sane to me. Lets get this in. |
Ah - minimal nitpick: @philippefichet could you please squash the two commits into one? |
@matthiasblaesing squash is done ;) (sorry for the delay) |
After 4 tries, travis is also happy. Lets get this in. Thank you. |
In the "bin" folder of "mvnd-0.7.1-windows-amd64" (latest current release), the list of files is as follows:
I take the liberty of adding the suffix ".cmd" when detecting under windows only