-
Notifications
You must be signed in to change notification settings - Fork 40
Fix test suite with libxml2 2.13.0 #87
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
base: master
Are you sure you want to change the base?
Conversation
|
@nwellnhof : hi! Sorry it took me long to review this pull-request. The first issue with it is that Test::Count annotations are missing: https://metacpan.org/pod/Test::Count . The second issue is that backward compatibility may have been broken too - including in https://metacpan.org/dist/XML-Grammar-ProductsSyndication . So I'm not sure this is the right direction. What do you think? |
I don't know how to fix this.
What do you mean exactly? Did you find a regression? |
|
Hi @nwellnhof !
Just add "# TEST" comments to account for the new test assertions.
Well, Perl 5 code that used to work in the past (like the one in the tests' suite or that of my ProductsSyndication module, ) stopped working. It now throws runtime errors. |
482715a to
bee8338
Compare
I added a "# TEST" comment for the new test and force-pushed.
The only things that have changed are error messages and bug fixes. I you found a regression, please open issue at https://gitlab.gnome.org/GNOME/libxml2/-/issues |
|
@nwellnhof : thanks! I'll try to investigate soon, |
Backport test suite fix from shlomif/perl-XML-LibXML#87
https://build.opensuse.org/request/show/1239251 by user pgajdos + anag+factory - Fix the testsuite with libxml2 version 2.13.0 and up: * github.com/shlomif/perl-XML-LibXML/pull/87 * Add perl-XML-LibXML-fix-testsuite-with-libxml2-2.13.patch (forwarded request 1239241 from pmonrealgonzalez)
|
Updated for the upcoming libxml2 2.14 release. |
|
I verified that clearlinux-pkgs/perl-XML-LibXML@7a00fd4#diff-1f9336e73dd201547c9d20e125f3c1f39548ee21b7cf94e09e8639978c3bd37f indeed fixes XML::LibXML on |
|
Can confirm also that this patch fixes the test failures using $ cpanm -L extlib --self-contained XML::LibXML
⋮
Failed 8/77 test programs. 9/2546 subtests failed.
make: *** [Makefile:1202: test_dynamic] Error 255
-> FAIL Installing XML::LibXML failed. See /root/.cpanm/work/1743536186.30209/build.log for details. Retry with --force to force install it.This is inside a Docker container so please don't freak out that I'm building it as root. Downloading and applying the patch from this PR: cd ~/.cpanm/work/1743536186.30209
cd XML-LibXML-2.0210
wget https://patch-diff.githubusercontent.com/raw/shlomif/perl-XML-LibXML/pull/87.diff
patch -p1 <87.diff
perl Makefile.PL # or, in my case: perl -Mlocal::lib=$HOME/src/extlib Makefile.PL
make install |
|
@shlomif when will this get merged and released? Getting this error trying to update a system to Cento 9 |
Line numbers are always enabled since libxml2 2.15.0.
|
fyi, I applied the proposed patch for the most recent Ubuntu build: still using 2.0207, I see these test failures: |
Make the test suite work with the upcoming 2.13.0 release of libxml2.