[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#586002: marked as done (initramfs-tools: should work around 'find -printf %Y' bug)



Your message dated Tue, 22 Jun 2010 13:34:21 +0200
with message-id <20100622113421.GA3562@stro.at>
and subject line Re: Bug#586002: initramfs-tools: should work around 'find -printf %Y' bug
has caused the Debian Bug report #586002,
regarding initramfs-tools: should work around 'find -printf %Y' bug
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 owner@bugs.debian.org
immediately.)


-- 
586002: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=586002
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: initramfs-tools
Version: 0.96.1
Severity: normal
Tags: patch
User: ubuntu-devel@lists.ubuntu.com
Usertags: origin-ubuntu ubuntu-patch maverick

The change to filter out looping or broken symlinks from the initramfs,
written by me for Ubuntu and applied to Debian for #575157, tickles a
bug in find (which I think also had an associated Debian bug, but I've
been unable to find it):

  https://savannah.gnu.org/bugs/index.php?29460

This was fixed in 4.5.8, which is currently only in experimental.  In
any case findutils is very deep in the dependency chain for upgrades and
I believe it would be a good idea to avoid requiring a new version of it
(I've had problems quite recently with strict dependencies on findutils
causing unresolvable dependency loops).  The following patch works
around this bug in a reasonably straightforward way.

diff --git a/mkinitramfs b/mkinitramfs
index 1c6ad28..e3b392e 100755
--- a/mkinitramfs
+++ b/mkinitramfs
@@ -304,8 +304,8 @@ fi
 
 # Remove any looping or broken symbolic links, since they break cpio.
 [ "${verbose}" = y ] && xargs_verbose="-t"
-find "${DESTDIR}" -type l -printf '%p %Y\n' | sed -n 's/ [LN]$//p' \
-	| xargs ${xargs_verbose:-} -rL1 rm -f
+(cd "${DESTDIR}" && find . -type l -printf '%p %Y\n' | sed -n 's/ [LN]$//p' \
+	| xargs ${xargs_verbose:-} -rL1 rm -f)
 
 [ "${verbose}" = y ] && echo "Building cpio ${outfile} initramfs"
 (

Thanks,

-- 
Colin Watson                                       [cjwatson@ubuntu.com]



--- End Message ---
--- Begin Message ---
somehow forgot to add the Closes to the commit message,
appending the uploaded initramfs-tools changes for 0.97 below.

thanks

-- 
maks

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Fri, 18 Jun 2010 12:28:04 +0200
Source: initramfs-tools
Binary: initramfs-tools
Architecture: source all
Version: 0.97
Distribution: unstable
Urgency: low
Maintainer: Debian kernel team <debian-kernel@lists.debian.org>
Changed-By: Michael Prokop <mika@debian.org>
Description: 
 initramfs-tools - tools for generating an initramfs
Closes: 396388 537139 582858 583695 585991 585992
Changes: 
 initramfs-tools (0.97) unstable; urgency=low
 .
   [ Martin Michlmayr ]
   * [e65ee48] Produce an error when root cannot be determined with
     MODULES=dep
   * [bb66fc2] hook-functions/init/scripts/local: add support for ubifs.
     (Closes: #582858) - thanks to Martin Michlmayr <tbm@cyrius.com>
 .
   [ Ferenc Wagner ]
   * [1d66ae1] scripts/nfs: cleanup retry logic
 .
   [ Colin Watson ]
   * [3054e3e] initramfs-tools: work around 'find -printf %Y' bug
 .
   [ maximilian attems ]
   * [a8a5ce4] script: Add update-initramfs.conf to bug report
   * [cc6077a] update-initramfs: Fix another unbound variable (Closes:
     #583695)
   * [b1f74e6] get_fstype: reference blkid in comment - thanks to
     Christoph Anton Mitterer <calestyo@scientia.net>
   * [eb93a7e] pre_mountroot(): reduce timeout to 30 seconds
   * [38563fe] scripts/functions: On panic change to tty1 if chvt around
     - thanks to Colin Watson <cjwatson@ubuntu.com>
   * [2031379] mkinitramfs: No point in hardcoding busybox path
   * [68c87cd] mkinitramfs: check syntax of boot and hook scripts
   * [ae02e4b] scripts/functions: beautify a bit reduce_satisfied()
   * [22d996c] maintainer doc: use git commands without the dash
   * [6147641] nfsmount: more small cleanups
   * [e967b03] mkinitramfs, lsinitramfs: Better error message for
     non-GNU getopt
   * Add lsinitramfs (to list content of an initramfs image) ([a39db63]
     [969f8fd] [fafede5] [2f3eb88])
 .
   [ Michael Prokop ]
   * [2a8c990] hook-functions/mkinitramfs/update-initramfs: consequently
     output error messages to stderr
   * [04b8619] init: display warning message if devtmpfs could not be
     mounted. - thanks to Ferenc Wagner <wferi@niif.hu>
   * [60afd2a] code cleanup: drop trailing whitespaces.
   * [ab28c77] code cleanup: consequently replace spaces in indentions
     with tabs to unify coding stile
   * [3a02c6f] code cleanup: consequently use "W:" for warnings.
   * [b1fff5e] add bash-completion for update-initramfs (Closes: #537139)
     - thanks to Stéphane Jourdois <stephane@jourdois.fr>
   * [5697c3b] hook-functions: replace awk calls with sed in
     dep_add_modules() (Closes: #585991) - thanks to Thorsten Glaser
     <tg@mirbsd.de>
   * [7afab22] debian/script: append /proc/mdstat (iff present) to
     bugreports
   * [5b565be] scripts/functions: allow hooks to abort build (Closes:
     #396388)
   * [528ba78] hook-functions: do not install ubi module by default via
     auto_add_modules()
   * [ecb8416] lsinitramfs: be more defensive against cmdline options
   * [2ff4ba2] scripts/functions: fix usage of test for script execution
   * [74f71c9] scripts/functions: fix another sh -n usage and fix typo
   * [91f5947] hook-functions: install virtio_pci module if
     /sys/bus/virtio is present and using modules=dep (Closes: #585992) -
     thanks to Vincent Danjean <vdanjean@debian.org>
   * [d25f610] slightly improve manpage lsinitramfs.8
   * Provide maintainer-notes.html, being "Maintainer documentation for
     initramfs-tools" ([7fc1ee5] [afafea4] [e506ee7] [d53a839] [6af23c2])
Checksums-Sha1: 
 1981a3c7cab77b8105606ef699b5a13c0d799dcf 948 initramfs-tools_0.97.dsc
 7d37400d055f5ba5300e6f735f3d5e34071fd155 78260 initramfs-tools_0.97.tar.gz
 1a24451b10f59779d1576587384e400b16177175 84888 initramfs-tools_0.97_all.deb
Checksums-Sha256: 
 e76167d657258eca81e156ebde6d3319855590acd8af1148dbb3055fe84e18fa 948 initramfs-tools_0.97.dsc
 682ab981a4a8ab489270e964ddd425beb29130a44b2a6b55e17a568f13ae0052 78260 initramfs-tools_0.97.tar.gz
 56eb56d472d0dd24c8f2fd030222586e258ec882b716f02d114865cef9c19639 84888 initramfs-tools_0.97_all.deb
Files: 
 4ef4180e40efded6fbfb0a76be43e3d1 948 utils optional initramfs-tools_0.97.dsc
 1d1d4885c8cdd5fd04dc899f91f9e361 78260 utils optional initramfs-tools_0.97.tar.gz
 10b97d7b5557007e5667d7a2f00c0539 84888 utils optional initramfs-tools_0.97_all.deb

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

iD8DBQFMG2rG2N9T+zficugRAvHWAJ9KKRsrGl/l3yYiN6vg0M7RO5lVHQCffvVO
zRoO9DJoNULpxZd7GbUuen0=
=SfJe
-----END PGP SIGNATURE-----




--- End Message ---

Reply to: