r48731 - in /trunk/eekboek/debian: changelog control rules

[joostvb at users.alioth.debian.org]

Author: joostvb
Date: Sun Dec 13 21:41:21 2009
New Revision: 48731

URL: http://svn.debian.org/wsvn/pkg-perl/?sc=1&rev=48731
Log:
disable pg-tests: we now succesfully build on systems with PostgreSQL installed

Modified:
    trunk/eekboek/debian/changelog
    trunk/eekboek/debian/control
    trunk/eekboek/debian/rules

Modified: trunk/eekboek/debian/changelog
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/eekboek/debian/changelog?rev=48731&op=diff
==============================================================================
--- trunk/eekboek/debian/changelog (original)
+++ trunk/eekboek/debian/changelog Sun Dec 13 21:41:21 2009
@@ -1,11 +1,9 @@
 eekboek (1.05.04-1) UNRELEASED; urgency=low
 
   TODO:
+  - test, test, test!  (especially ebgui)
   - stick the lib/EB/DB/Sqlite.pm patch in a sane
     patch system
-  - fix FTBFS and re-enable a full 'make test' in debian/rules
-    for now, t/91_ivp_sqlite.t is disabled in debian/rules.
-    It fails.
   - lintian (and lintian -I):
     example-interpreter-not-absolute
     E: eekboek: perl-module-in-core-directory usr/share/perl/5.10.1/EB.pm
@@ -13,8 +11,9 @@
   - d/copyright: 2005-2009, Squirrel Consultancy
     (to be honest, I haven't check all the other modules)
   - the new eekboek.desktop should probably be installed
-  - how about the new script/*?
-    ebshell is mentioned in README.Debian
+  - mention ebgui in README.Debian.  upstream suggests to run it
+    using an explicit path; we however install it in the default
+    $PATH.
   - d/control: the version in "perl (>= 5.8.8-7etch1)" is not needed
   - might eekboek-mode.el be interesting?
   - debian/TODO: some items seem outdated, some others irrelevant
@@ -52,8 +51,11 @@
   * debian/rules: install manpages under /usr/share/man, not /usr/man.
   * debian/control: add libarchive-zip-perl, libterm-readline-gnu-perl,
     libhtml-parser-perl (HTML::Entities), libdbd-sqlite3-perl (>= 1.13),
-    libdbd-pg-perl (>= 1.41) to build depends as suggested by upstream at
-    http://www.eekboek.nl/install/build.html.
+    to build depends as suggested by upstream at
+    http://www.eekboek.nl/install/build.html.  We don't need libdbd-pg-perl
+    (>= 1.41) since we've disabled the pg-tests.
+  * debian/rules: disable tests 80_db_postgresql.t, 91_ivp_sqlite.t,
+    92_ivp_postgres.t. For some reason these fail.
 
  -- Nathan Handler <nhandler at ubuntu.com>  Wed, 04 Nov 2009 22:30:20 -0600
 

Modified: trunk/eekboek/debian/control
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/eekboek/debian/control?rev=48731&op=diff
==============================================================================
--- trunk/eekboek/debian/control (original)
+++ trunk/eekboek/debian/control Sun Dec 13 21:41:21 2009
@@ -5,7 +5,7 @@
 Build-Depends-Indep: perl (>= 5.8.8-7etch1), libipc-run3-perl (>= 0.034),
  perl (>= 5.10.0) | libmodule-build-perl (>= 0.26), libdbi-perl (>= 1.40),
  libfile-homedir-perl, libarchive-zip-perl, libterm-readline-gnu-perl,
- libhtml-parser-perl, libdbd-sqlite3-perl (>= 1.13), libdbd-pg-perl (>= 1.41)
+ libhtml-parser-perl, libdbd-sqlite3-perl (>= 1.13)
 Maintainer: Debian Perl Group <pkg-perl-maintainers at lists.alioth.debian.org>
 Uploaders: Joost van Baal <joostvb at debian.org>, Nathan Handler <nhandler at ubuntu.com>
 Standards-Version: 3.8.3

Modified: trunk/eekboek/debian/rules
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/eekboek/debian/rules?rev=48731&op=diff
==============================================================================
--- trunk/eekboek/debian/rules (original)
+++ trunk/eekboek/debian/rules Sun Dec 13 21:41:21 2009
@@ -3,21 +3,20 @@
 %:
 	dh $@
 
-# remove failing test; upstream wants us to use Build.PL, not Makefile.PL
+# upstream wants us to use Build.PL, not Makefile.PL: we override dh's default
+
+# remove failing tests.
+# The tests 80_db_postgresql.t and 92_ivp_postgres.t need
+# libdbd-pg-perl (>= 1.41) build depends
 override_dh_auto_configure:
-	rm -f t/91_ivp_sqlite.t
+	rm -f t/80_db_postgresql.t t/91_ivp_sqlite.t t/92_ivp_postgres.t
 	perl Build.PL INSTALLDIRS=vendor create_packlist=0
 
-# upstream wants us to use ./Build, not "make test"
 override_dh_auto_test:
 	./Build test
 
-# upstream wants us to use ./Build, not make
 # use debian/eekboek. not dh's default debian/tmp
 override_dh_auto_install:
 	./Build install --destdir `pwd`/debian/eekboek --prefix /usr --install_path libdoc=/usr/share/man/man3 --install_path bindoc=/usr/share/man/man1
 	rm -f debian/eekboek/usr/share/perl5/EB/DB/Postgres.pm
 
-# disable 'make test' by uncommenting this
-## override_dh_auto_test:
-## 	: