[New-bugs-announce] [issue35501] "make coverage" should use leak coverage flags to third party C extensions

[STINNER Victor]

New submission from STINNER Victor <vstinner at redhat.com>:

"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 <report at bugs.python.org>
<https://bugs.python.org/issue35501>
_______________________________________