[libx11-windowhierarchy-perl] 01/01: Corrected packaging issues pointed out by Gregor Hermann

[Dima Kogan]

This is an automated email from the git hooks/post-receive script.

dkogan-guest pushed a commit to branch master
in repository libx11-windowhierarchy-perl.

commit 6bc43228f301b26ad1a894fe35b4825200c0cc2a
Author: Dima Kogan <dima at secretsauce.net>
Date:   Thu Jan 16 11:11:37 2014 -0800

    Corrected packaging issues pointed out by Gregor Hermann
    
    - better versioning of libx11-protocol-perl dependencies
    - better versioning of perl dependencies
    - better versioning of debhelper dependencies
    - running tests inside an xvfb
---
 debian/changelog | 24 +-----------------------
 debian/control   | 12 +++++++-----
 debian/rules     |  4 ++++
 3 files changed, 12 insertions(+), 28 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index bce8cd5..d2e3de9 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,26 +1,4 @@
-libx11-windowhierarchy-perl (0.004-1) UNRELEASED; urgency=low
-
-  TODO:
-  - debian/control:
-    + libx11-protocol-perl (in B-D-I and Depends) doesn't need a version,
-      there's nothing older in the archive (oven oldstable has a new enough
-      version). probably a dh-make-perl artefact; "cme fix dpkg" helps
-    + Build-Depends-Indep: "perl (>= 5.13.11)" is also a dh-make-perl
-      artefact, this should be "perl, perl (>= 5.13.11) | libtest-simple-perl
-      (>= 0.98)"
-      (the reason being Test::More 0.98 in META.*)
-    + Build-Depends: "debhelper (>= 9.20120312)" is only needed for arch:any
-      packages to get the hardening flags while compiling, for arch:all
-      packages debhelper 8 is enough. (9 is fine if you prefer it but
-      9.20120312 is a bit overspecific)
-    + B-D-I again: adding libtest-pod-perl would enable an additional test
-  - the package FTBFS (in my cowbuilder chroot):
-
-    Can't connect to display `unix:0': No such file or directory at /usr/share/perl5/X11/Protocol.pm line 2264.
-    # Looks like your test exited with 2 before it could output anything.
-    t/hierarchy.t ...
-
-    good news: it works with xvfb --> http://paste.debian.net/76599/
+libx11-windowhierarchy-perl (0.004-1) unstable; urgency=low
 
   * Initial Release. (Closes: #735256)
 
diff --git a/debian/control b/debian/control
index d31d8fb..442cbab 100644
--- a/debian/control
+++ b/debian/control
@@ -3,9 +3,10 @@ Section: perl
 Priority: optional
 Maintainer: Debian Perl Group <pkg-perl-maintainers at lists.alioth.debian.org>
 Uploaders: Dima Kogan <dima at secretsauce.net>
-Build-Depends: debhelper (>= 9.20120312)
-Build-Depends-Indep: libx11-protocol-perl (>= 0.56),
- perl (>= 5.13.11)
+Build-Depends: debhelper (>=8)
+Build-Depends-Indep: libx11-protocol-perl,
+                     perl, perl (>= 5.13.11) | libtest-simple-perl (>= 0.98),
+                     libtest-pod-perl, xauth, xvfb
 Standards-Version: 3.9.5
 Vcs-Browser: http://anonscm.debian.org/gitweb/?p=pkg-perl/packages/libx11-windowhierarchy-perl.git
 Vcs-Git: git://anonscm.debian.org/pkg-perl/packages/libx11-windowhierarchy-perl.git
@@ -13,8 +14,9 @@ Homepage: https://metacpan.org/release/X11-WindowHierarchy
 
 Package: libx11-windowhierarchy-perl
 Architecture: all
-Depends: ${misc:Depends}, ${perl:Depends},
- libx11-protocol-perl (>= 0.56)
+Depends: ${misc:Depends},
+         ${perl:Depends},
+         libx11-protocol-perl
 Description: Perl module for retrieving the current X11 window hierarchy
  X11::WindowHierarchy provides a couple of helper functions based on
  X11::Protocol for extracting the current window hierarchy. You can easily get
diff --git a/debian/rules b/debian/rules
index 2d33f6a..c242220 100755
--- a/debian/rules
+++ b/debian/rules
@@ -2,3 +2,7 @@
 
 %:
 	dh $@
+
+# The tests need an X server. This provides one even inside a chroot
+override_dh_auto_test:
+	xvfb-run -a dh_auto_test

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-perl/packages/libx11-windowhierarchy-perl.git