On 06/11/2025 12: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]> > --- > appveyor.yml | 2 +- > m4/absolute-header.m4 | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-)
Thanks Ilya. I confirmed it's working in my AppVeyor. Acked-by: Kevin Traynor <[email protected]> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
