Bug#535364: syntax error in /etc/apt/apt.conf.d/90debsums

2009-07-02 Thread Filippo Giunchedi
reopen 535364
thanks

On Thu, Jul 02, 2009 at 11:42:43AM +1200, Francois Marier wrote:
 Hi Filippo,
 
 On 2009-07-02 at 00:46:42, Filippo Giunchedi wrote:
  in config file /etc/apt/apt.conf.d/90debsums there's a syntax error of
  trailing :: after DPkg::Post-Invoke, trivial patch:
 
 I don't think that this is a syntax error, have a look at the manpage for
 apt.conf(5):
 
All of the APT tools take a -o option which allows an arbitrary
configuration directive to be specified on the command line. The
syntax is a full option name (APT::Get::Assume-Yes for instance)
followed by an equals sign then the new value of the option. Lists
can be appended too by adding a trailing :: to the list name.

That's right, but it is referring to either trailing :: or {} (which gives
implicit ::), in other words:

$ cat 90debsums 
DPkg::Post-Invoke:: { if [ -x /usr/bin/debsums ]; then /usr/bin/debsums
--generate=nocheck -sp /var/cache/apt/archives; fi; };
$ apt-config dump | grep -i debsums
DPkg::Post-Invoke if [ -x /usr/bin/debsums ]; then /usr/bin/debsums
--generate=nocheck -sp /var/cache/apt/archives; fi;

notice the 

if you either remove {} or :: it correctly lists a single ::

$ cat 90debsums 
DPkg::Post-Invoke { if [ -x /usr/bin/debsums ]; then /usr/bin/debsums
--generate=nocheck -sp /var/cache/apt/archives; fi; };
$ apt-config dump | grep -i debsums
DPkg::Post-Invoke:: if [ -x /usr/bin/debsums ]; then /usr/bin/debsums
--generate=nocheck -sp /var/cache/apt/archives; fi;

$ cat 90debsums 
DPkg::Post-Invoke:: if [ -x /usr/bin/debsums ]; then /usr/bin/debsums
--generate=nocheck -sp /var/cache/apt/archives; fi;
go...@clamp:/etc/apt/apt.conf.d$ apt-config dump | grep -i debsums
DPkg::Post-Invoke:: if [ -x /usr/bin/debsums ]; then /usr/bin/debsums
--generate=nocheck -sp /var/cache/apt/archives; fi;

whether the apt parser doesn't barf on this is another matter :)

filippo
-- 
Filippo Giunchedi - http://esaurito.net - 0x6B79D401

Endian little hate we
-- Anonymous (?)



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#535364: syntax error in /etc/apt/apt.conf.d/90debsums

2009-07-02 Thread Francois Marier
tags 535364 + pending
thanks

On 2009-07-02 at 10:14:39, Filippo Giunchedi wrote:
 That's right, but it is referring to either trailing :: or {} (which gives
 implicit ::)

Thanks for reopening the bug and clarifying this. The documentation wasn't
overly clear on that.

I've fixed it in git and I assume that the new maintainer will include as
part of his first upload.

Cheers,
Francois



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#535364: syntax error in /etc/apt/apt.conf.d/90debsums

2009-07-01 Thread Filippo Giunchedi
Package: debsums
Version: 2.0.46
Severity: minor

Hi,
in config file /etc/apt/apt.conf.d/90debsums there's a syntax error of
trailing :: after DPkg::Post-Invoke, trivial patch:

--- debsums.postinst2009-06-26 06:31:32.0 +0200
+++ /tmp/debsums.postinst   2009-07-02 00:45:48.0 +0200
@@ -20,7 +20,7 @@
 configure)
 db_get debsums/apt-autogen || true
 if [ $RET = true ]; then
-echo 'DPkg::Post-Invoke:: { if [ -x /usr/bin/debsums ]; then 
/usr/bin/debsums --generate=nocheck -sp /var/cache/apt/archives; fi; };' 
$APT_CONF
+echo 'DPkg::Post-Invoke { if [ -x /usr/bin/debsums ]; then 
/usr/bin/debsums --generate=nocheck -sp /var/cache/apt/archives; fi; };' 
$APT_CONF
 else
 rm -f $APT_CONF
 fi



-- System Information:
Debian Release: squeeze/sid
  APT prefers unstable
  APT policy: (990, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.29.4 (SMP w/2 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages debsums depends on:
ii  debconf [debconf-2.0] 1.5.26 Debian configuration management sy
ii  perl  5.10.0-23  Larry Wall's Practical Extraction 

debsums recommends no packages.

debsums suggests no packages.

-- debconf information:
  debsums/apt-autogen: true
  debsums/croncheck: never



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org