[Build-common-hackers] Bug#378013: cdbs: list-missing never actually report that all files were installed

[Rémi Denis-Courmont]

Package: cdbs
Version: 0.4.44
Severity: minor


	Hello,

the list-missing utility rule does not detect the case whereby all files
from debian/tmp were actually installed somewhere.
"All files were installed into debian/<package>" will actually only be
printed if, not only all files were installed, but there is no extra
files into debian/<package> that is not in debian/tmp. In practice, that
is unfortunately almost always the case because dh_installdocs and
dh_installchangelogs install files into /usr/share/doc in addition to
"make install".

The problem lies in the regular expression used in this shell exerpt
(from list-missing):

diff -u debian/cdbs-install-list debian/cdbs-package-list | egrep '^-'

That matches the initial "---" line from the unified diff file, even if
there is no missing file. An alternative could be:

diff -u debian/cdbs-install-list debian/cdbs-package-list | { read; cat;
} | egrep '^-'

The "read" would ensure that the first line, which is always "--- ..."
be ignored.

Regards

-- System Information:
Debian Release: testing/unstable
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'testing')
Architecture: i386 (i686)
Shell:  /bin/sh linked to /bin/dash
Kernel: Linux 2.6.17.4
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)

cdbs depends on no packages.

Versions of packages cdbs recommends:
ii  autotools-dev                 20060223.1 Update infrastructure for config.{
ii  debhelper                     5.0.37.2   helper programs for debian/rules

-- no debconf information