Re: [ovs-dev] [PATCH] appveyor: Move from MinGW 32bit to msys64.
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 > Signed-off-by: Alin Gabriel Serdean 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 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
Re: [ovs-dev] [PATCH] appveyor: Move from MinGW 32bit to msys64.
Looking forward to that patch :) Thank you for migrating the CI to msys! Acked-by: Alin Gabriel Serdean Signed-off-by: Alin Gabriel Serdean > > On 12 Feb 2024, at 13:43, Ilya Maximets 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
Re: [ovs-dev] [PATCH] appveyor: Move from MinGW 32bit to msys64.
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
