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

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

Author: dleidert-guest
Date: Fri Feb  8 08:51:32 2008
New Revision: 1028

URL: http://svn.debian.org/wsvn/debichem/?sc=1&rev=1028
Log:
Added a TODO file containing some comments and explanations about the recent
work on debian/rules, which you can find at http://debian.wgdd.de/temp/rules.
Please test and feel free to add comments to this file, answer questions,
whatever ... :)


Added:
    unstable/gromacs/debian/TODO

Added: unstable/gromacs/debian/TODO
URL: http://svn.debian.org/wsvn/debichem/unstable/gromacs/debian/TODO?rev=1028&op=file
==============================================================================
--- unstable/gromacs/debian/TODO (added)
+++ unstable/gromacs/debian/TODO Fri Feb  8 08:51:32 2008
@@ -1,0 +1,46 @@
+TODO for gromacs
+----------------
+
+* debian/rules: why LIBSUFFIX is used creating mpich targets?
+
+* 04_autoconf_mpisuffix.dpatch: AC_ARG_ENABLE normally only has two values:
+  yes and no - AC_ARG_WITH or AR_ARG_VAR (which is probably much easier to use
+  for this) should be used:
+
+  [..]
+  AC_ARG_VAR([LIBSUFFIX_ADD], [append the value to the MPI library names])
+  [..]
+  if test -n "$LIBSUFFIX_ADD"; then
+    LIBSUFFIX="${LIBSUFFIX}_${LIBSUFFIX_ADD}"
+  fi
+  [..]
+
+  ./configure ...blah... LIBSUFFIX=openmpi MPICC=mpicc.openmpi
+
+  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)
+
+* debian/rules is some kind of hard to read. It can be written easier (already
+  done locally - test before commit):
+
+  e.g.: remove those clean-calls between configure && make && make install by
+
+  (mkdir -p build/basic; cd build/basic; ../../configure $(COMMON_CONFIG_PARAMS))
+  (mkdir -p build/basic-dp; cd build/basic-dp; ../../configure $(COMMON_CONFIG_PARAMS) --enable-double)
+  $(MAKE) -C build/basic
+  $(MAKE) -C build/basic-dp
+  $(MAKE) -C build/basic install DESTDIR=$(CURDIR)/debian/gromacs
+  $(MAKE) -C build/basic-dp install DESTDIR=$(CURDIR)/debian/gromacs
+
+  (also use DESTDIR and not prefix nor exec_prefix for installation)
+
+* some link calls should be done by dh_link and gzipping manpages is better
+  done using dh_installman
+
+* 40_missing_cpp.dpatch, 10_manpages.dpatch: Can these be applied upstream?
+  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.
+