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

bpo-46932: Update bundled libexpat to 2.4.7 #31736

Merged
merged 2 commits into from Mar 7, 2022
Merged

bpo-46932: Update bundled libexpat to 2.4.7 #31736

merged 2 commits into from Mar 7, 2022

Conversation

zooba
Copy link
Member

@zooba zooba commented Mar 7, 2022

Copy link
Contributor

@hartwork hartwork left a comment

The pyexpatns.h part should probably not be dropped, please see the comment below.

The PR is likely good once it passes this Dockerfile of mine (and it does not, yet):

# Copyright (c) 2022 Sebastian Pipping <sebastian@pipping.org>
# Licensed under the Apache License version 2.0

FROM alpine
RUN apk add --update \
            diffutils \
            git \
            sed \
        && \
    git clone --depth 1 https://github.com/python/cpython cpython-main \
        && \
    ( cd cpython-main && git rev-parse HEAD ) \
        && \
    git clone --depth 1 --branch bpo-46932 https://github.com/zooba/cpython zooba-bpo-46932 \
        && \
    ( cd zooba-bpo-46932 && git rev-parse HEAD ) \
        && \
    git config --global advice.detachedHead false \
        && \
    git clone --depth 1 --branch R_2_4_6 https://github.com/libexpat/libexpat libexpat_2_4_6 \
        && \
    git clone --depth 1 --branch R_2_4_7 https://github.com/libexpat/libexpat libexpat_2_4_7 \
        && \
    diff -r -u libexpat_2_4_6/expat/lib/ cpython-main/Modules/expat/ | tee 2-4-6.diff \
        && \
    diff -r -u libexpat_2_4_7/expat/lib/ zooba-bpo-46932/Modules/expat/ | tee 2-4-7.diff \
        && \
    sed -e '/^Only in /d' -e '/^\(+++\|---\) /d' -e '/^diff /d' -i 2-4-6.diff 2-4-7.diff \
        && \
    diff -u 2-4-6.diff 2-4-7.diff \
        && \
    echo 'Diff is good.'

Modules/expat/expat_external.h Outdated Show resolved Hide resolved
@zooba zooba merged commit 176835c into python:main Mar 7, 2022
12 checks passed
@miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Mar 7, 2022

Thanks @zooba for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7, 3.8, 3.9, 3.10.
🐍🍒🤖

@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented Mar 7, 2022

GH-31738 is a backport of this pull request to the 3.10 branch.

miss-islington added a commit to miss-islington/cpython that referenced this issue Mar 7, 2022
(cherry picked from commit 176835c)

Co-authored-by: Steve Dower <steve.dower@python.org>
@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented Mar 7, 2022

GH-31739 is a backport of this pull request to the 3.9 branch.

miss-islington added a commit to miss-islington/cpython that referenced this issue Mar 7, 2022
(cherry picked from commit 176835c)

Co-authored-by: Steve Dower <steve.dower@python.org>
@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented Mar 7, 2022

GH-31740 is a backport of this pull request to the 3.8 branch.

@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented Mar 7, 2022

GH-31741 is a backport of this pull request to the 3.7 branch.

miss-islington added a commit to miss-islington/cpython that referenced this issue Mar 7, 2022
(cherry picked from commit 176835c3d5c70f4c1b152cc2062b549144e37094)

Co-authored-by: Steve Dower <steve.dower@python.org>
miss-islington added a commit to miss-islington/cpython that referenced this issue Mar 7, 2022
(cherry picked from commit 176835c)

Co-authored-by: Steve Dower <steve.dower@python.org>
@zooba zooba deleted the bpo-46932 branch Mar 7, 2022
miss-islington added a commit that referenced this issue Mar 7, 2022
(cherry picked from commit 176835c)

Co-authored-by: Steve Dower <steve.dower@python.org>
miss-islington added a commit that referenced this issue Mar 7, 2022
(cherry picked from commit 176835c)

Co-authored-by: Steve Dower <steve.dower@python.org>
ned-deily pushed a commit that referenced this issue Mar 7, 2022
(cherry picked from commit 176835c)

Co-authored-by: Steve Dower <steve.dower@python.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants