➜
This issue trackerhas been migrated toGitHub, and is currentlyread-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.
Created on2018-10-18 00:26 bygregory.p.smith, last changed2022-04-11 14:59 byadmin. This issue is nowclosed.
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 9939 | merged | gregory.p.smith,2018-10-18 00:42 | |
| PR 9940 | merged | miss-islington,2018-10-18 01:11 | |
| PR 9941 | merged | miss-islington,2018-10-18 01:11 | |
| Messages (7) | |||
|---|---|---|---|
| msg327919 -(view) | Author: Gregory P. Smith (gregory.p.smith)* | Date: 2018-10-18 00:26 | |
These lines used to exist inModules/expat/expat_external.h:/* Namespace external symbols to allow multiple libexpat version to co-exist. */#include "pyexpatns.h"https://github.com/python/cpython/commit/5dc3f23b5fb0b510926012cb3732dae63cddea60#diff-3afaf7274c90ce1b7405f75ad825f545removed them during an expat upgrade.This causes link time conflicts when embedding Python using its own expat in an application that also uses libexpat from the C/C++ side on its own. | |||
| msg327920 -(view) | Author: Gregory P. Smith (gregory.p.smith)* | Date: 2018-10-18 00:28 | |
Not a release blocker as most users probably do not run into this problem, but the pyexpatns.h mechanics should be restored. | |||
| msg327921 -(view) | Author: Gregory P. Smith (gregory.p.smith)* | Date: 2018-10-18 00:37 | |
This appears to not have been shipped in a release yet. It is new in 3.6.7. cc'ing ned daily to see if he wants to include the fix (the PR is trivial, coming ASAP).I don't have a good feel for how this impacts the real world or not.We noticed because we embed CPython in larger applications that use a different copy of libexpat on their own. | |||
| msg327924 -(view) | Author: Gregory P. Smith (gregory.p.smith)* | Date: 2018-10-18 01:10 | |
New changeset9d4712bc8f26bf1d7e626b53ab092fe030bcd68d by Gregory P. Smith in branch 'master':bpo-35011: Restore use of pyexpatns.h in libexpat (GH-9939)https://github.com/python/cpython/commit/9d4712bc8f26bf1d7e626b53ab092fe030bcd68d | |||
| msg327927 -(view) | Author: miss-islington (miss-islington) | Date: 2018-10-18 02:05 | |
New changeset4bfecb9298d447d5599ea76f3f68f772c38b8fd0 by Miss Islington (bot) in branch '3.6':[3.6]bpo-35011: Restore use of pyexpatns.h in libexpat (GH-9939) (GH-9941)https://github.com/python/cpython/commit/4bfecb9298d447d5599ea76f3f68f772c38b8fd0 | |||
| msg327928 -(view) | Author: miss-islington (miss-islington) | Date: 2018-10-18 02:06 | |
New changeset35ae99d7b394af0ce01460f7bccd7449a82289ad by Miss Islington (bot) in branch '3.7':[3.7]bpo-35011: Restore use of pyexpatns.h in libexpat (GH-9939) (GH-9940)https://github.com/python/cpython/commit/35ae99d7b394af0ce01460f7bccd7449a82289ad | |||
| msg327934 -(view) | Author: Benjamin Peterson (benjamin.peterson)* | Date: 2018-10-18 04:54 | |
Sorry for breaking that, and thanks for the fix!I'm curious, though, why are you still using the embedded expat rather than linking everything against the same expat? | |||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:59:07 | admin | set | github: 79192 |
| 2018-10-18 04:54:49 | benjamin.peterson | set | messages: +msg327934 |
| 2018-10-18 02:07:24 | gregory.p.smith | set | status: open -> closed resolution: fixed stage: patch review -> commit review |
| 2018-10-18 02:06:34 | miss-islington | set | messages: +msg327928 |
| 2018-10-18 02:05:52 | miss-islington | set | nosy: +miss-islington messages: +msg327927 |
| 2018-10-18 01:11:11 | miss-islington | set | pull_requests: +pull_request9292 |
| 2018-10-18 01:11:03 | miss-islington | set | pull_requests: +pull_request9291 |
| 2018-10-18 01:10:51 | gregory.p.smith | set | messages: +msg327924 |
| 2018-10-18 00:42:05 | gregory.p.smith | set | keywords: +patch stage: needs patch -> patch review pull_requests: +pull_request9290 |
| 2018-10-18 00:37:53 | gregory.p.smith | set | priority: normal -> deferred blocker nosy: +ned.deily messages: +msg327921 |
| 2018-10-18 00:28:03 | gregory.p.smith | set | assignee:gregory.p.smith type: compile error components: + Build, Extension Modules title: Update to expat removed the pyexpatns.h, causing link time symbol conflicts vs other versions in an application -> expat: Restore the use of pyexpatns.h to avoid link time conflicts vs other versions nosy: +benjamin.peterson messages: +msg327920 stage: needs patch |
| 2018-10-18 00:26:15 | gregory.p.smith | create | |