[Debichem-commits] r1029 - /unstable/gromacs/debian/TODO

[dleidert-guest at users.alioth.debian.org]

Author: dleidert-guest
Date: Fri Feb  8 09:04:46 2008
New Revision: 1029

URL: http://svn.debian.org/wsvn/debichem/?sc=1&rev=1029
Log:
* some more thoughts


Modified:
    unstable/gromacs/debian/TODO

Modified: unstable/gromacs/debian/TODO
URL: http://svn.debian.org/wsvn/debichem/unstable/gromacs/debian/TODO?rev=1029&op=diff
==============================================================================
--- unstable/gromacs/debian/TODO (original)
+++ unstable/gromacs/debian/TODO Fri Feb  8 09:04:46 2008
@@ -20,8 +20,13 @@
   the patch should further drop the changed line numbers in echo calls (I
   don't think, we need them).
 
-* CFLAGS is currently not given to the build, so noopt will have no effect
-  and debugging symbols are not stored (missing -g)
+* CFLAGS was not given to the build, so noopt had no effect and debugging
+  symbols were not stored (missing -g)
+
+  -> current way: define an empty CFLAGS to let the configure script determine
+     the CFLAGS
+  -> depending on DEB_BUILD_OPTS, we define CFLAGS and this will lead to
+     CFLAGS being given to configure, which then skips its own tests
 
 * debian/rules is some kind of hard to read. It can be written easier (already
   done locally - test before commit):
@@ -37,6 +42,13 @@
 
   (also use DESTDIR and not prefix nor exec_prefix for installation)
 
+  currently leads to:
+  1 configure target
+  1 build target
+  split install target
+
+  should build and configure be split into separate targets too?
+
 * some link calls should be done by dh_link and gzipping manpages is better
   done using dh_installman
 
@@ -44,3 +56,9 @@
   What about 04_autoconf_mpisuffix.dpatch? Can we talk to upstream to apply
   this one as well? Probably other packagers will benefit from it too.
 
+* 04_autoconf_mpisuffix.dpatch: triggers a rerun of the autotools chain
+  -> comment the configure.ac stuff?
+  -> ask upstream to add the AM_MAINTAINER_MODE macro?
+
+* TODO: manpage errors - triggered by leading points followed by a space and a
+  word - should be fixed upstream too