On 11/6/25 5:49 PM, Eelco Chaudron wrote: > > > On 6 Nov 2025, at 13:14, Ilya Maximets wrote: > >> MSYS2 is trying to move closer to cygwin and as part of that effort >> they changed reporting of the system triplets from msys to cygwin: >> >> https://www.msys2.org/news/#2025-06-20-replacing-x86_64-pc-msys-with-x86_64-pc-cygwin >> >> This broke our builds in AppVeyor, since detection of absolute paths >> for system headers is now using the wrong path separator list: >> >> checking absolute name of <stdio.h>... "" >> checking absolute name of <string.h>... "" >> >> Fix by adding cygwin to the list for Windows-style paths. With that, >> we can properly detect the headers again: >> >> checking absolute name of <stdio.h>... >> "C:\\Program Files (x86)\\Windows >> Kits\\10\\include\\10.0.22000.0\\ucrt\\stdio.h" >> checking absolute name of <string.h>... >> "C:\\Program Files (x86)\\Windows >> Kits\\10\\include\\10.0.22000.0\\ucrt\\string.h" >> >> We used the 'Previous' variant of the image for AppVeyor before to >> avoid this issue, but now even the previous image has the new MSYS2 >> and CI is fully broken without the fix. >> >> Signed-off-by: Ilya Maximets <[email protected]> > > Change looks good to me! Let's hope it's fixed for good now :) > > Acked-by: Eelco Chaudron <[email protected]> >
Thanks, Mike, Kevin and Eelco! Applied. Best regards, Ilya Maximets. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
