This is an automated email from the git hooks/post-receive script.

guillem pushed a commit to branch master
in repository dpkg.

View the commit online:
https://git.dpkg.org/cgit/dpkg/dpkg.git/commit/?id=58fb0496a61053fec3c65a3460f1a47c93d31ff1

commit 58fb0496a61053fec3c65a3460f1a47c93d31ff1
Author: Guillem Jover <guil...@debian.org>
AuthorDate: Fri Aug 24 21:20:14 2018 +0200

    build: Disable C optimization levels when configuring for code coverage
    
    Optimizations mess up the coverage reporting, as the reslting object
    layout changes substantialy to the point of it not matching what is
    being actually covered by the test suite.
---
 debian/changelog    | 1 +
 m4/dpkg-coverage.m4 | 4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index 2d9d9e0e4..6ebe703fc 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -239,6 +239,7 @@ dpkg (1.19.1) UNRELEASED; urgency=medium
     - Distribute the man.stamp from VPATH.
     - Preserve timestamps when distributing man pages.
     - Add a GitLab CI configuration file.
+    - Disable C optimization levels when configuring for code coverage.
   * Packaging:
     - Install update-alternatives policykit-1 file.
     - Add Breaks to libdpkg-perl against pkg-kde-tools (<< 0.15.28~), as
diff --git a/m4/dpkg-coverage.m4 b/m4/dpkg-coverage.m4
index 4e14b736a..6cc097d0e 100644
--- a/m4/dpkg-coverage.m4
+++ b/m4/dpkg-coverage.m4
@@ -29,8 +29,8 @@ AC_DEFUN([DPKG_CODE_COVERAGE], [
        AC_MSG_ERROR([missing genhtml, which is required for C coverage 
support])
      ])
 
-     CFLAGS="$CFLAGS --coverage"
-     LDFLAGS="$LDFLAGS --coverage"
+     CFLAGS="$CFLAGS --coverage -O0"
+     LDFLAGS="$LDFLAGS --coverage -O0"
 
      AC_MSG_CHECKING([for Devel::Cover perl module])
      AS_IF([$($PERL -e "require Devel::Cover;" 2>/dev/null)], [

-- 
Dpkg.Org's dpkg

Reply via email to