[Pkg-ganeti-devel] Fwd: Initial debian/ directory

[Leonardo Rodrigues de Mello]

---------- Forwarded message ----------
From: Leonardo Rodrigues de Mello <l at lmello.eu.org>
Date: 03/10/2007 17:55
Subject: Re: [Pkg-ganeti-devel] Initial debian/ directory
To: Guido Trotter <ultrotter at debian.org>


Tested, and fixed some issues:
first issue, the os-search-path in rules was not correct:
chmod a+x /trab/lmello/ganeti/ganeti/ganeti-1.2rev188/./configure
cd . && CC="cc" CXX="g++" CFLAGS="-g -Wall -O2" CXXFLAGS="-g -Wall
-O2" CPPFLAGS="" LDFLAGS=""
/trab/lmello/ganeti/ganeti/ganeti-1.2rev188/./configure
--build=i486-linux-gnu --prefix=/usr --includedir="\${prefix}/include"
--mandir="\${prefix}/share/man" --infodir="\${prefix}/share/info"
--sysconfdir=/etc --localstatedir=/var
--libexecdir="\${prefix}/lib/ganeti" --disable-maintainer-mode
--disable-dependency-tracking --srcdir=.  --prefix=/usr
--localstatedir=/var --with-export-dir=/var/lib/ganeti/export
--with-os-search-path=/srv/ganeti/os, /usr/local/lib/ganeti/os,
/usr/local/share/ganeti/os, /usr/lib/ganeti/os, /usr/share/ganeti/os
configure: WARNING: you should use --build, --host, --target
configure: WARNING: invalid host type: /usr/local/lib/ganeti/os,
configure: WARNING: you should use --build, --host, --target
configure: WARNING: invalid host type: /usr/local/share/ganeti/os,
configure: WARNING: you should use --build, --host, --target
configure: WARNING: invalid host type: /usr/lib/ganeti/os,
configure: WARNING: you should use --build, --host, --target
configure: WARNING: invalid host type: /usr/share/ganeti/os
---------------------------------------------------------------------------------------------
I believe that the email of ganeti packaging team can be this
maillinglist, there was one mispelling of maintenance in the file
copyright and solved lot almost all warnings from lintian. There is
just one warning now, about manpage of ganeti-master.

here is the diff:
/debian# svn diff
=====BEGIN HERE============
# svn diff debian/
Index: debian/control
===================================================================
--- debian/control      (revisão 3)
+++ debian/control      (cópia de trabalho)
@@ -1,7 +1,7 @@
 Source: ganeti
 Section: admin
 Priority: extra
-Maintainer: Debian Ganeti Team
+Maintainer: Debian Ganeti Team <pkg-ganeti-devel at lists.alioth.debian.org>
 Uploaders: Guido Trotter <ultrotter at debian.org>, Iustin Pop
<iusty at k1024.org>, Leonardo Rodrigues de Mello <l at lmello.eu.org>
 Build-Depends: cdbs, debhelper (>= 5), docbook-utils
 Standards-Version: 3.7.2
@@ -9,7 +9,7 @@
 Package: ganeti
 Architecture: any
 Depends: ${shlibs:Depends}, ${misc:Depends}, python (>= 2.4), lvm2,
ssh, bridge-utils, iproute, iputils-arping, fping,
python-twisted-core, python-pyopenssl,  openssl, mdadm
-Recommends: drbd0.7-utils, xen-linux-system-2.6.18-5-xen-vserver-686
| xen-linux-system-2.6.18-4-xen-686 |
xen-linux-system-2.6.18-5-xen-686 |
xen-linux-system-2.6.18-4-xen-vserver-686
+Recommends: xen-linux-system-2.6.18-5-xen-vserver-686 |
xen-linux-system-2.6.18-4-xen-686 | xen-linux-system-2.6.18-5-xen-686
| xen-linux-system-2.6.18-4-xen-vserver-686
 Suggests: drbd0.7-module-source, drbd0.7-utils
 Description: Cluster-based virtualization management software
  Ganeti is a virtual server cluster management software tool built on top
Index: debian/copyright
===================================================================
--- debian/copyright    (revisão 3)
+++ debian/copyright    (cópia de trabalho)
@@ -1,7 +1,7 @@
 This package was debianized by Leonardo Rodrigues de Mello <l at lmello.eu.org> on
 Thu, 30 Aug 2007 10:38:20 -0300.

-It was then taken over for official Debian maintainance by the Debian
Ganeti Team.
+It was then taken over for official Debian  maintenance by the Debian
Ganeti Team.

 It was downloaded from http://code.google.com/p/ganeti/

Index: debian/rules
===================================================================
--- debian/rules        (revisão 3)
+++ debian/rules        (cópia de trabalho)
@@ -8,11 +8,7 @@
   --prefix=/usr \
   --localstatedir=/var \
   --with-export-dir=/var/lib/ganeti/export \
-  --with-os-search-path=/srv/ganeti/os,\
-                        /usr/local/lib/ganeti/os,\
-                       /usr/local/share/ganeti/os,\
-                       /usr/lib/ganeti/os,\
-                       /usr/share/ganeti/os
+  --with-os-search-path=/srv/ganeti/os,/usr/local/lib/ganeti/os,/usr/local/share/ganeti/os,/usr/lib/ganeti/os,/usr/share/ganeti/os
 DEB_DH_INSTALLINIT_ARGS = --no-start

 clean::
@@ -22,4 +18,6 @@
 build/ganeti::
        cp doc/examples/ganeti.cron debian/ganeti.cron
        cp doc/examples/ganeti.initd debian/ganeti.init
-
+binary-post-install/ganeti::
+       find debian/$(cdbs_curpkg) -name '*.py[co]' -print0 | xargs -0 rm -f --
+       find debian/$(cdbs_curpkg) -name '*.py' -print | grep -v
_autoconf.py | xargs chmod a+x --
====END HERE ============
There one issue with ganeti autotools build system, related to initd
script.. this script is inside directory doc/examples, Actualy, when
someone run make install inside doc/examples/ it will put ganeti.initd
inside /usr/bin/ganeti.initd
I believe that this script must be placed inside /usr/share/doc/ganeti/.
I don't know what would be the best aproach to do that using the
actual structure, but i had solved this issue with the following
patch. to ganeti upstream:
# svn diff doc/examples/Makefile.am
Index: doc/examples/Makefile.am
===================================================================
--- doc/examples/Makefile.am    (revisão 188)
+++ doc/examples/Makefile.am    (cópia de trabalho)
@@ -1,9 +1,9 @@
 EXTRA_DIST = ganeti.initd.in ganeti.cron
 CLEANFILES = $(nodist_bin_SCRIPTS)

-nodist_bin_SCRIPTS = ganeti.initd
-
-all-local: ganeti.initd
+dist_doc_DATA = ganeti.initd \
+               ganeti.cron
+all-local: ganeti.initd
 ganeti.initd: ganeti.initd.in
        sed \
          -e "s#@PREFIX@#$(prefix)#g" \


-- 
Leonardo Rodrigues de Mello
jabber: l at lmello.eu.org