Your message dated Sun, 05 May 2013 04:02:35 +0000
with message-id <[email protected]>
and subject line Bug#700200: fixed in debhelper 9.20130504
has caused the Debian Bug report #700200,
regarding debhelper: incomplete Depends lists due to deprecated find(1) usage
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
700200: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=700200
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: debhelper
Version: 9.20120909
Severity: important
Tags: patch

Dear Maintainer,

Midweek, some of our packages in SprezzOS (a Debian derivative) started to be
built with incomplete or entirely missing Depends lines. You can read the saga
here:

     https://www.sprezzatech.com/bugs/show_bug.cgi?id=638

I traced it down this morning to line 118 of dh_shlibdeps, which was not
properly enumerating executables. You'll find there the find(1) predicate
"-perm +111". Consult the find manpage:


       -perm +mode
              Deprecated,  old way of searching for files with any of the
per‐
              mission bits in mode set.  You should use -perm  /mode  instead.
              Trying to use the `+' syntax with symbolic modes will yield
sur‐
              prising results.  For example, `+u+x' is a valid  symbolic  mode
              (equivalent to +u,+x, i.e. 0111) and will therefore not be
eval‐
              uated as -perm +mode but instead as  the  exact  mode  specifier
              -perm  mode  and so it matches files with exact permissions 0111
              instead of files with any execute bit set.  If  you  found  this
              paragraph  confusing,  you're  not alone - just use -perm /mode.
              This form of the -perm test  is  deprecated  because  the  POSIX
              specification  requires  the  interpretation of a leading `+' as
              being part of a symbolic mode, and so we switched to  using  `/'
              instead.

In SprezzOS, we're using findutils 4.5.11 rather than Debian Unstable's 4.4.2.
With this newer findutils, the -perm +111 test generates an empty list. I
downgraded to 4.4.2 and verified that this fixed the issue. I then applied the
attached patch to debhelper, and verified that packages were now properly built
using both versions of findutils. We now consider this bug closed.

Debian does not appear to see this problem yet. However, at some point you will
upgrade to some unknown version of findutils which exposes the issue. Since the
test as stands is deprectated, and the proper test exists in both versions, and
functions properly in both, I strongly urge you to apply my patch.

Thanks!

--rigorously, nick
Hacker in Charge, SprezzOS Project



-- System Information:
Debian Release: 1 (von Neumann)
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 3.7.6 (SMP w/8 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 debhelper depends on:
ii  binutils    2.23.1-SprezzOS1
ii  dpkg        1.16.9-SprezzOS1
ii  dpkg-dev    1.16.9-SprezzOS1
ii  file        5.11-SprezzOS1
ii  html2text   1.3.2a-15
ii  man-db      2.6.3-SprezzOS1
ii  perl        5.16.2-SprezzOS1
ii  po-debconf  1.0.16+nmu2

debhelper recommends no packages.

Versions of packages debhelper suggests:
pn  dh-make  <none>

-- no debconf information
diff -ur debhelper-pristine/dh_shlibdeps debhelper-9.20120909/dh_shlibdeps
--- debhelper-pristine/dh_shlibdeps	2013-02-09 15:15:56.406329132 -0500
+++ debhelper-9.20120909/dh_shlibdeps	2013-02-09 15:16:57.252994702 -0500
@@ -129,7 +129,7 @@
 	if (defined($dh{EXCLUDE_FIND}) && $dh{EXCLUDE_FIND} ne '') {
 		$find_options="! \\( $dh{EXCLUDE_FIND} \\)";
 	}
-	foreach my $file (split(/\n/,`find $tmp -type f \\( -perm +111 -or -name "*.so*" -or -name "*.cmxs" \\) $find_options -print`)) {
+	foreach my $file (split(/\n/,`find $tmp -type f \\( -perm -111 -or -name "*.so*" -or -name "*.cmxs" \\) $find_options -print`)) {
 		# Prune directories that contain separated debug symbols.
 		next if $file=~m!^\Q$tmp\E/usr/lib/debug/(lib|lib64|usr|bin|sbin|opt|dev|emul)/!;
 		# TODO this is slow, optimize. Ie, file can run once on

--- End Message ---
--- Begin Message ---
Source: debhelper
Source-Version: 9.20130504

We believe that the bug you reported is fixed in the latest version of
debhelper, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to [email protected],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Joey Hess <[email protected]> (supplier of updated debhelper package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [email protected])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Format: 1.8
Date: Sat, 04 May 2013 23:32:27 -0400
Source: debhelper
Binary: debhelper
Architecture: source all
Version: 9.20130504
Distribution: unstable
Urgency: low
Maintainer: Joey Hess <[email protected]>
Changed-By: Joey Hess <[email protected]>
Description: 
 debhelper  - helper programs for debian/rules
Closes: 525821 680339 683551 683557 683941 684689 684895 686696 690378 690399 
691398 691557 692208 693100 700200 701233 703262 703515 706314
Changes: 
 debhelper (9.20130504) unstable; urgency=low
 .
   * dh_shlibdeps: Warn if -V flag is passed, to avoid it accidentially being
     used here rather than in dh_makeshlibs. Closes: #680339
   * dh_lintian: Source overrides doc improvement. Closes: #683941
   * dh_installmime: No longer makes maintainer scripts run update-mime and
     update-mime-database, that is now handled by triggers. Closes: #684689
     Thanks, Charles Plessy
   * python distutils buildsystem: Propagate failure of pyversions.
     Closes: #683551 Thanks, Clint Byrum
   * python distutils buildsystem: When checking if a version of python is
     installed, don't trust the presense of the executable, as
     a python2.X-minimal package may provide it while not having
     distutils installed. Closes: #683557, #690378
   * dh_icons: Improve documentation. Closes: #684895
   * Improve -X documentation. Closes: #686696
   * Support installing multiple doc-base files which use the same doc-id.
     Closes: #525821
   * dh_installdocs: Support having the same document id in different binary
     packages built from the same source.
     Closes: #525821 Thanks, Don Armstrong
   * dh_installdeb: Avoid unnecessary is_udeb tests. Closes: #691398
   * Updated German man page translation. Closes: #691557, #706314
   * dh_installinit: Support systemd.
     Closes: #690399 Thanks, Michael Stapelberg
   * Updated French man page translation. Closes: #692208
   * dh_icons: Reword description. Closes: #693100
   * Avoid find -perm +mode breakage caused by findutils 4.5.11,
     by instead using -perm /mode. Closes: #700200
   * cmake: Configure with -DCMAKE_BUILD_TYPE=RelWithDebInfo
     Closes: #701233
   * dh_auto_test: Avoid doing anything when cross-compiling. Closes: #703262
   * dh_testdir: Fix error message. Closes: #703515
Checksums-Sha1: 
 d85b2648ca8d1fc90ad43584d3bd5875dee44db6 1577 debhelper_9.20130504.dsc
 da3e094d2b0a61f6aeadfe1f0ed5c6bdcea67877 470050 debhelper_9.20130504.tar.gz
 3c1da0c1f75877cafa598b677b5c1e9366560b51 706414 debhelper_9.20130504_all.deb
Checksums-Sha256: 
 9dd245024b2be9230bcd6569c617d2dae39e19c206cb639c72e4e252c3c86d3b 1577 
debhelper_9.20130504.dsc
 7d6ee2657430afcb477396c32c4bc170aa7fae97332045aa8f4ebec889051f01 470050 
debhelper_9.20130504.tar.gz
 08450bd3fbae0bcef04e60d299c8557634e35f027a446a454ab574f8948d2419 706414 
debhelper_9.20130504_all.deb
Files: 
 6823c687eb955536e3d5d32b69a7dd41 1577 devel optional debhelper_9.20130504.dsc
 b5ea99c845e78cecbc89d25484a46b42 470050 devel optional 
debhelper_9.20130504.tar.gz
 fe4f489a57e830c66c6b2fbac672cdf4 706414 devel optional 
debhelper_9.20130504_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (GNU/Linux)

iQIVAwUBUYXVcckQ2SIlEuPHAQj8sw//R3bheMCm70k4D33QJ9Xc8wVHlcYgmY2f
Bi7ChTYcCtgyazS7yZmQ759Sh+m9vn11v/qqzVNKsxhIUnjjHFLWWfpcdCdn//Zp
6aAmRN0OT6ovLYFBAbuB80EBeSOE41QjKLUV1M7IU7DoFi+kCWWFHpv2Pni1XP/B
c7GqNAOI9CDKXEivyrgcv+UIA1PGCGz3Pk8J3ZF8Jy5GT2zelmvczj00uDVj/jUf
qWjQRirHrpOpmH8gcImhmlTS02WD6vAfNYuCa98HCh2o6QxWLx9tvgGRkudvHUr4
U4rT6ITAakVELQyVLbG/C9AhnHSMW96OT5ew834w3jcbPcXu9qMeTyUQyo/I9Od8
Wtipn28JoG9D04omMQMktXV8rdu1jfY8splBkow+kE7gIkjCpk6WkwBBd5XqvWAq
Xihf7eijRM1E2GbbEACg4pnVa1YynoZqhQhHf1V3bhoxYHtRbWHfV+KwXNznob/p
kOuOuftGpYANvP66E8kI9IDu612x95+Wn2G1KhLhvuX9CPXr/JDe3+fsYpvpGmJz
dZlVD787+LjEc3p5+sx+IGngNEjUa8CecIsJhir1Z6rZ9mGIPkVZZeEWBAXTi+kz
c8h4elA2ebSeB9qTHX5iXNbgtxmdkBZiddhb5dyF89DA25FQHcgm0GQu9Pmocido
lkNNVbB1DvU=
=YmiJ
-----END PGP SIGNATURE-----

--- End Message ---

Reply via email to