Package: pgbadger
Version: 1.0-1
Severity: normal
Tags: patch

Hello,

Thanks for making pgbadger available in debian !

I was surprised the package has no dependency field. I guess it should
at least depend on perl ?

    $ apt-cache show pgbadger | grep -c Depen
    0

Also, the doc mentions pgbadger can parse CSV logs is Text::CSV_XS is
available. So I guess this module should be at least suggested to users?

The attached patch fixes these 2 details.

Cheers,
Marc

-- System Information:
Debian Release: 7.0
  APT prefers testing
  APT policy: (700, 'testing'), (500, 'testing-proposed-updates'), (500, 
'stable'), (300, 'unstable'), (100, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 3.2.0-4-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

-- no debconf information
diff -r -u pgbadger-1.0.original/debian/control pgbadger-1.0/debian/control
--- pgbadger-1.0.original/debian/control	2013-04-08 17:02:33.000000000 +0200
+++ pgbadger-1.0/debian/control	2013-04-08 17:16:03.892450383 +0200
@@ -8,7 +8,8 @@
 
 Package: pgbadger
 Architecture: all
-Depends: ${misc:Depends}
+Depends: ${perl:Depends}
+Suggests: libtext-csv-xs-perl
 Description: Fast PostgreSQL log analysis report
  pgBadger is a PostgreSQL log analyzer build for speed with fully detailed
  reports from your PostgreSQL log file. It's a single and small script written

Reply via email to