STINNER Victor <vstin...@redhat.com> added the comment:
New changeset 92f90242994652d6b7afe0a2c8a61cf2bccc8c32 by Ned Deily (Miss Islington (bot)) in branch '3.7': bpo-35257: fix broken BLDSHARED - needs LDFLAGS too (GH-11297) https://github.com/python/cpython/commit/92f90242994652d6b7afe0a2c8a61cf2bccc8c32 Since Ned pushed a new fix, I ran again my tests on the 3.7 branch with ./configure CC=clang --with-lto --prefix /opt/py37 --enable-shared: (1) 4 => ok (2) 0, False => ok (3) 0 => ok Good! It works (in my tests). -- "make sharedmods" runs "LDSHARED=$BLDSHARED (...) ./python setup.py build" "make libpython(...)" uses $(BLDSHARED) to link libpython. Modules/makesetup uses BLDSHARED to build C extensions of the stdlib. Example from generated Makefile: "Modules/posix$(EXT_SUFFIX): Modules/posixmodule.o; $(BLDSHARED) Modules/posixmodule.o -o Modules/posix$(EXT_SUFFIX) (...)" distutils has a customize_compiler() function which uses LDSHARED from Makefile (can be overriden by environment variables): it doesn't use BLDSHARED. It seems like BLDSHARED is only used to build Python itself and stdlib modules. So I agree that it's ok to add PY_LDFLAGS_NODIST flags to BLDSHARED. ---------- _______________________________________ Python tracker <rep...@bugs.python.org> <https://bugs.python.org/issue35257> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
