-
-
Notifications
You must be signed in to change notification settings - Fork 904
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ci: upstream libxml2 testing: windows and ruby_memcheck coverage #2868
Conversation
@MSP-Greg Sorry to have to ask for help. I'm having a bit of trouble getting autotools/libtool to work properly on Windows. The error I'm seeing is, from the
despite trying to install these packages in the - uses: ruby/setup-ruby-pkgs@v1
with:
ruby-version: "3.2"
apt-get: "autogen libtool shtool"
brew: "automake autogen libtool shtool"
mingw: "_upgrade_ autotools"
bundler-cache: true
bundler: latest I've tried a few variations of what packages I specify, under both Am I missing something obvious? Can you make a suggestion at what I should look at next? |
Sorry for the delay. I believe config.guess and config.sub need to be downloaded, I think ruby/ruby does so. Not sure about the other items. Build systems are not something I have much knowledge of. Both files are 'small', so they could be added to the Windows 'build tool packages' that are installed by setup-ruby. Not sure where to put them. I checked MSY2 and MSFT vcpkg, and neither have updated to libxml2 2.11.0, so looking at their build scripts probably won't be very helpful... |
@MSP-Greg Thanks for responding, I appreciate your time. I have to apologize for not explaining better what I am having trouble with -- mostly because I'm not sure what's going wrong, so thanks for your patience. I've tried explicitly installing libtool which contains the files being logged as missing. And they're still not being found. Do you know: is there some sort of My next step is going to be digging into where pacman installs the files and what the paths are in the environment, but the feedback loops are slow here so I was hoping you'd spot something silly that I'm doing or not doing. |
@MSP-Greg Apologies again, after some more digging I don't think this is related to the mingw packages, I think the problem is likely something in the Nokogiri build system that's causing the |
I think For the info on what packages are installed with setup-ruby, see https://github.com/ruby/setup-msys2-gcc/actions/workflows/windows-build-tools.yml The most recent workflow run shows jobs for MSYS2 and the various build archives that are available. The 'msys2 tools' jobs show that As to the other jobs, they're pairs of MINGW64 & UCRT64 gcc tools and ruby build dependencies, with the Back to this issue, C:/msys64/usr/share/libtool/build-aux
The ruby/ruby build scripts copy some of the files into the src folder, in a JFYI, I use nokogiri daily when updating https://msp-greg.github.io/. I also always use ruby master/head/trunk, so I never use the pre-compiled gem. This code has been in place for years. I normally had the |
instead of invoking `env` on the command line
@MSP-Greg Just closing the loop with you -- see flavorjones/mini_portile#126 for the underlying fix. It was a matter of converting a windows path to a posix path so the |
3152067
to
83f1b6d
Compare
to enable us to do upstream libxml2 builds on windows
83f1b6d
to
ec2c97b
Compare
and don't wait for the normal builds to finish
8c5376d
to
b1a74c2
Compare
for latest libxml2 2.11
b1a74c2
to
37d8196
Compare
What problem is this PR intended to solve?
libxml2 2.11.0 is failing windows builds, and I think I probably could have caught the problem earlier if I was running the upstream builds on windows. This PR adds that integration testing.
libxml2 is also leaking memory, so this PR also adds
ruby_memcheck
testing on the upstream builds to try to detect these problems earlier.