From LRN: Other tests that caught my attention were tools/tests, which appeared to be crashing. Turns out, it was an assertion failure, in each one of them (by the way, calling abort() on W32 is not a good idea; it's better to call DebugBreak(), then ExitProcess ()). Apparently, they are all trying to mkdtemp ("/tmp/..."), which obviously fails, since W32 doesn't support POSIX paths.
Created attachment 77367 [details] [review] Use a semi-colon as the path separator on Windows
LRN, I've your PATH separator patch from bug 63046. I just changed the line that says '#ifdef OS_WIN32'. Could you test it?
Compiles, passes the tests (i replaced my patch with your version). And why wouldn't it? In configure.ac you define OS_WIN32 based on host triplet *-*-mingw*. Only mingw has that triplet (cygwin is "*-*-cygwin, i think), so effect is the same.
Created attachment 77374 [details] [review] More compatible path munging and handling code
Thanks for testing that. I've folded it into a far bigger patch which moves all the path munging code into a central location, so we can test it for sanity. Could you review and test the attached (much bigger) patch?
Complies, passes all tests.
Attachment 77374 [details] pushed as a63311a - More compatible path munging and handling code
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.