Package: debian-policy
Severity: wishlist

The attached patch should mostly speak for itself.  I think it's great
that a lot of packages support DEB_BUILD_OPTIONS=debug now, but if the
program is compiled with optimization, it's very difficult to debug.

Another alternative is to make a whole different option "noopt" or
something; I'd accept that as well.

--- debian-policy-3.5.6.1/policy.sgml   2002-03-14 13:17:48.000000000 -0500
+++ debian-policy-3.5.6.1.hacked/policy.sgml    2002-08-17 22:36:06.000000000 
-0400
@@ -5362,10 +5362,12 @@
          If the environment variable <tt>DEB_BUILD_OPTIONS</tt>
          contains the string <tt>debug</tt>, compile the software
          with debugging information (usually this involves adding the
-         <tt>-g</tt> flag to <tt>CFLAGS</tt>).  This allows the
-         generation of a build tree with debugging information.  If
-         the environment variable <tt>DEB_BUILD_OPTIONS</tt> contains
-         the string <tt>nostrip</tt>, do not strip the files at
+         <tt>-g</tt> and the <tt>-O0</tt> flags to <tt>CFLAGS</tt>).
+         This allows the generation of a build tree with debugging
+         information; the reduced optimization ensures that the
+         debugging tools can easily analyze the program.  If the
+         environment variable <tt>DEB_BUILD_OPTIONS</tt> contains the
+         string <tt>nostrip</tt>, do not strip the files at
          installation time.  This allows one to generate a package
          with debugging information included.<footnote>
            <p>
@@ -5384,7 +5386,7 @@
          test for either condition; you will probably have to massage
          this example in order to make it work for your package.
          <example compact="compact">
-CFLAGS = -O2 -Wall
+CFLAGS = -Wall
 INSTALL = install
 INSTALL_FILE    = $(INSTALL) -p    -o root -g root  -m  644
 INSTALL_PROGRAM = $(INSTALL) -p    -o root -g root  -m  755
@@ -5392,7 +5394,9 @@
 INSTALL_DIR     = $(INSTALL) -p -d -o root -g root  -m  755
 
 ifneq (,$(findstring debug,$(DEB_BUILD_OPTIONS)))
-CFLAGS += -g
+CFLAGS += -g -O0
+else
+CFLAGS += -O2
 endif
 ifeq (,$(findstring nostrip,$(DEB_BUILD_OPTIONS)))
 INSTALL_PROGRAM += -s




Reply via email to