[Python-checkins] bpo-35257: fix broken BLDSHARED - needs LDFLAGS too (GH-11297)
Miss Islington (bot)
webhook-mailer at python.org
Sun Dec 23 15:48:05 EST 2018
https://github.com/python/cpython/commit/3d4b4b80f290e622b05ca219ad6dabc07b49421a
commit: 3d4b4b80f290e622b05ca219ad6dabc07b49421a
branch: 3.7
author: Miss Islington (bot) <31488909+miss-islington at users.noreply.github.com>
committer: GitHub <noreply at github.com>
date: 2018-12-23T12:48:00-08:00
summary:
bpo-35257: fix broken BLDSHARED - needs LDFLAGS too (GH-11297)
`BLDSHARED` needs to have both `LDFLAGS` and `LDFLAGS_NODIST`, not just `LDFLAGS_NODIST`; `PY_CORE_LDFLAGS` provides both. For example, as it stands now with just `LDFLAGS_NODIST`, macOS universal builds are broken as the necessary `-arch` flags are no longer passed to the standard library extension module link step from `setup.py` resulting in extension modules being single architecture only.
https://bugs.python.org/issue35257
(cherry picked from commit 44a3ee07e30e18d83e2730c093d8b0e930f0a06c)
Co-authored-by: Ned Deily <nad at python.org>
files:
M Makefile.pre.in
diff --git a/Makefile.pre.in b/Makefile.pre.in
index d727288db823..2d2e11f17209 100644
--- a/Makefile.pre.in
+++ b/Makefile.pre.in
@@ -155,7 +155,7 @@ CONFINCLUDEPY= $(CONFINCLUDEDIR)/python$(LDVERSION)
SHLIB_SUFFIX= @SHLIB_SUFFIX@
EXT_SUFFIX= @EXT_SUFFIX@
LDSHARED= @LDSHARED@ $(PY_LDFLAGS)
-BLDSHARED= @BLDSHARED@ $(PY_LDFLAGS_NODIST)
+BLDSHARED= @BLDSHARED@ $(PY_CORE_LDFLAGS)
LDCXXSHARED= @LDCXXSHARED@
DESTSHARED= $(BINLIBDEST)/lib-dynload
More information about the Python-checkins
mailing list