mocobeta commented on pull request #391: URL: https://github.com/apache/lucene/pull/391#issuecomment-946544474
@janhoy @dweiss Commands with `./` do not work on Windows. Yesterday I just tested it on my Windows OS; I saw Command Prompt and PowerShell do not support `./gradlew`. After I noticed this comment in the file, I deleted the comment... sorry for the noise. https://github.com/apache/lucene/blob/6c21862a552cccbb8509e4383ac8c6d10c68137f/dev-tools/scripts/smokeTestRelease.py#L43-L45 I think it could be labor to make the scripts OS-agnostic. As for Windows, instead of fully supporting Windows perhaps we could test it on WSL2 then throw away Cygwin? I have little experience with it, but it seems to work just as plain Ubuntu and it's easier to install than Cygwin (its I/O performance was terrible a few years ago, but it should have improved...). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
