On 2/12/24 14:05, Alin Serdean wrote: > Looking forward to that patch :) > Thank you for migrating the CI to msys! > > Acked-by: Alin Gabriel Serdean <[email protected]> > Signed-off-by: Alin Gabriel Serdean <[email protected]>
Thanks! Applied. I'll clean up my OpenSSL patches and send them out. :) Best regards, Ilya Maximets. > >> >> On 12 Feb 2024, at 13:43, Ilya Maximets <[email protected]> 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 [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
