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