Skip to content
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

fix: Don't build uwsgi with xml support. #1157

Merged
merged 2 commits into from
Nov 15, 2024

Commits on Nov 13, 2024

  1. fix: Don't build uwsgi with xml support.

    When uwsgi runs with xml support, it throws the following error on
    startup with the latest version of edx-platform:
    
    ```
    xmlsec.InternalError: (-1, 'lxml & xmlsec libxml2 library version mismatch')
    ```
    
    See xmlsec/python-xmlsec#320 for more details
    on this.
    
    Essentially, the uWSGI wheel is built against a different version of
    libxml2 and it causes this version mismach error when trying to load
    xmlsec.  The xml support in uWSGI is only needed for runing with xml
    configuration files, which tutor does not do.  Following the guidance of
    the above issue, I updated the `openedx` Dockerfile to no longer compile
    with any xml support as a part of the build. The time to build uWSGI was
    only slightly more than building from cache so I'm not concerned about
    major slowdowns in the build time for un-cached builds.
    feanil committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    8aba298 View commit details
    Browse the repository at this point in the history

Commits on Nov 14, 2024

  1. Configuration menu
    Copy the full SHA
    20c1d99 View commit details
    Browse the repository at this point in the history