Looking forward to that patch :) Thank you for migrating the CI to msys! Acked-by: Alin Gabriel Serdean <aserd...@ovn.org> Signed-off-by: Alin Gabriel Serdean <aserd...@ovn.org>
> > On 12 Feb 2024, at 13:43, Ilya Maximets <i.maxim...@ovn.org> wrote: > > On 2/9/24 21:47, Ilya Maximets wrote: >> AppVeyor is planning to remove support for MinGW 32bit soon. And we >> had a couple of incidents where it wasn't available already, so we >> moved to a 'Previous' image. >> >> Move to msys64 instead. >> >> While at it making the CI scripts a little nicer, moving the non-Windows >> parts of the preparation and build to separate files. >> >> MSYS2 has its own version of python. However, we do not support >> building on Windows with non-Windows python build. The main issue is >> the delimiter symbol in PYTHONPATH. In Windows version it has to be >> ';', while the python supplied with MSYS2 uses ':' as on Linux, while >> we detect Windows and pass ';' during the build. Renaming the binary, >> so the Windows version is used. >> >> Additionally switched to Python 3.12, 3.7 reached EoL some time back, >> though it's still available in AppVeyor. >> >> The stderr has to be redirected to stdout for scripts, because any >> message on stderr is treated as fatal failure by powershell. >> >> Scripts are running with 'set -e', so a failure of individual >> commands will fail the script. >> >> The OpenSSL download is still failing, but it is out of scope for >> this change. > > For the record, I have a fix for OpenSSL ready, including the > permission fixes with icacls, but it depends on changes made > in this patch. > > Best regards, Ilya Maximets. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
