New submission from STINNER Victor <[email protected]>:
"make coverage" modifies CFLAGS and LIBS, Makefile.pre.in:
coverage:
@echo "Building with support for coverage checking:"
$(MAKE) clean profile-removal
$(MAKE) @DEF_MAKE_RULE@ CFLAGS="$(CFLAGS) -O0 -pg -fprofile-arcs
-ftest-coverage" LIBS="$(LIBS) -lgcov"
CFLAGS_NODIST should be used instead here. I'm not sure about LIBS: do we need
LIBS_NODIST, as we have CFLAGS_NODIST? LIBS_NODIST would be used for Python and
C extensions of the stdlib, but not for third-party C extensions: not used by
distutils.
See also bpo-35257: "Avoid leaking linker flags into distutils: add
PY_LDFLAGS_NODIST".
----------
components: Build
messages: 331850
nosy: vstinner
priority: normal
severity: normal
status: open
title: "make coverage" should use leak coverage flags to third party C
extensions
versions: Python 3.8
_______________________________________
Python tracker <[email protected]>
<https://bugs.python.org/issue35501>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
