On Sun, 26 Jun 2022 12:31:22 GMT, Nikita Gubarkov <d...@openjdk.org> wrote:

> Use temp directory for autoconf on both WSL1 and WSL2

For me it fails since [this 
commit](https://github.com/JetBrains/JetBrainsRuntime/commit/2d4af2255feb2eaeca533424f8cba3ec0945d757).
Turned out earlier it incorrectly defined my environment as wsl2, though I was 
building on wsl1, but everything worked fine. And then it started failing when 
defining my environment as wsl1.

I don't fully understand why it fails, it successfully builds conftest.exe, 
then runs and it exits with 139 code (SIGSEGV). I couldn't reproduce it 
manually, but it fails every time in configure script. And though it's 100% 
reproducible for me, my colleagues weren't able to reproduce it.

Given that I don't know why this helps, I just took the codepath which worked 
for me before. Anyway this patch shouldn't break anything, as it just makes 
WSL1 use %TEMP% instead of current working directory which are both on Windows 
FS. At least on those machines where this issue wasn't reproducible, it still 
works with this fix.

-------------

PR: https://git.openjdk.org/jdk/pull/9289

Reply via email to