Issue 18164: Embedding Python doc incorrectly refers to LINKFORSHARED (original) (raw)
This issue has been migrated to GitHub: https://github.com/python/cpython/issues/62364
classification
Title: | Embedding Python doc incorrectly refers to LINKFORSHARED | ||
---|---|---|---|
Type: | Stage: | resolved | |
Components: | Documentation | Versions: | Python 3.3, Python 3.4, Python 2.7 |
process
Status: | closed | Resolution: | fixed |
---|---|---|---|
Dependencies: | Superseder: | ||
Assigned To: | ned.deily | Nosy List: | E.Money, eric.araujo, ned.deily, pitrou, python-dev, wichert |
Priority: | normal | Keywords: | patch |
Created on 2013-06-07 20:52 by ned.deily, last changed 2022-04-11 14:57 by admin. This issue is now closed.
Files | |||
---|---|---|---|
File name | Uploaded | Description | Edit |
issue18164_3x.patch | ned.deily,2013-06-13 00:37 | 3x doc patch | review |
issue18164-27.patch | ned.deily,2013-06-13 00:37 | 27 doc patch |
Messages (6) | ||
---|---|---|
msg190779 - (view) | Author: Ned Deily (ned.deily) * ![]() |
Date: 2013-06-07 20:52 |
This is in reference to section 5.6 of the "Extending and Embedding Python" doc. This has come up recently where third-party projects are using LINKEDFORHARED instead of the results of python-config when linking an embedded Python into their projects. See discussion in Issue3588. Also, the updates to section 5.6 made for Python 3 should be backported to the Python 2.7 doc. http://docs.python.org/3/extending/embedding.html#compiling-and-linking-under-unix-like-systems http://docs.python.org/2/extending/embedding.html#linking-requirements | ||
msg191034 - (view) | Author: egallager (E.Money) | Date: 2013-06-12 15:44 |
For reference, this originally came from the following MacPorts ticket: https://trac.macports.org/ticket/39223#comment:14 | ||
msg191044 - (view) | Author: Éric Araujo (eric.araujo) * ![]() |
Date: 2013-06-12 19:38 |
I think it was Antoine who committed the patch originally. | ||
msg191052 - (view) | Author: Ned Deily (ned.deily) * ![]() |
Date: 2013-06-13 00:37 |
On further review, I think the 3.x documentation is pretty good as is. I propose one change to it: getting another config var to help emphasize "the configuration values that you will want to combine together". The proposed 2.7 patch backports the whole 3.x section including that change. | ||
msg191807 - (view) | Author: Roundup Robot (python-dev) ![]() |
Date: 2013-06-24 21:24 |
New changeset e18b92bae4d6 by Ned Deily in branch '2.7': Issue #18164: Backport the more detailed embedding compile-and-link section http://hg.python.org/cpython/rev/e18b92bae4d6 New changeset 4a114b0db866 by Ned Deily in branch '3.3': Issue #18164: Clarify the embedding docs regarding link options. http://hg.python.org/cpython/rev/4a114b0db866 New changeset 262689e0fa2a by Ned Deily in branch 'default': Issue #18164: merge from 3.3 http://hg.python.org/cpython/rev/262689e0fa2a | ||
msg191808 - (view) | Author: Ned Deily (ned.deily) * ![]() |
Date: 2013-06-24 21:26 |
Patches committed (with consistent use of get_config_var as suggested by Antoine) for 2.7.5, 3.3.3, and 3.4.0 docs. |
History | |||
---|---|---|---|
Date | User | Action | Args |
2022-04-11 14:57:46 | admin | set | github: 62364 |
2013-06-24 21:26:58 | ned.deily | set | status: open -> closedresolution: fixedmessages: + stage: commit review -> resolved |
2013-06-24 21:24:11 | python-dev | set | nosy: + python-devmessages: + |
2013-06-13 00:37:32 | ned.deily | set | files: + issue18164-27.patch |
2013-06-13 00:37:20 | ned.deily | set | files: + issue18164_3x.patchkeywords: + patchmessages: + stage: needs patch -> commit review |
2013-06-12 19:38:11 | eric.araujo | set | nosy: + pitrou, eric.araujomessages: + |
2013-06-12 15:44:14 | E.Money | set | nosy: + E.Moneymessages: + |
2013-06-07 20:56:29 | wichert | set | nosy: + wichert |
2013-06-07 20:52:24 | ned.deily | create |