Giter Site home page Giter Site logo

Comments (4)

nielsdos avatar nielsdos commented on May 28, 2024 1

Thanks for the report. The failures come from both the evolution of the expat library, and because some tests assume libxml2 behaviour with respect to error handling. There are small differences in how errors are handled and reported which cause visible differences in the tests. The differences are not bugs. The solution I went with was to split the tests.

I'm unsure about the support status of expat in php.

Still works, just not tested on a regular basis.

But when building, it links against both libxml2 and expat, and I'm unsure what the implications are.

The XML extension links against expat, but since you still have other XML extension enabled (e.g. the DOM extension), they link against libxml2. This has no negative impact.

Those docs also link to a very old expat webpage, and the notes about building expat are probably obsolete. I think it would be good to update and clarify those docs.

I've made a PR to update the expat library URLs too: php/doc-base#117

(The setlocale failures seem unrelated to expat/xml, they also appear when building with default options)

Seems harmless, it's configuration related.

from php-src.

nielsdos avatar nielsdos commented on May 28, 2024 1

We could add it to the nightly workflow.
Either as a small seperate task or by compiling the xml extension shared twice and loading each of them once to test. Probably the first option is the most easy one. Cc @iluuu1994 what do you think?

from php-src.

iluuu1994 avatar iluuu1994 commented on May 28, 2024 1

@nielsdos That makes sense to me.

from php-src.

hartwork avatar hartwork commented on May 28, 2024

@nielsdos is there a chance that the CI could be made to also cover compilation and test execution with --with-expat so that it's impossible to break without noticing in the future?

from php-src.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.