Debian Bug report logs - #852569
cloud-initramfs-tools: Update udevadm path

version graph

Package: cloud-initramfs-tools; Maintainer for cloud-initramfs-tools is Debian Cloud <debian-cloud@lists.debian.org>;

Reported by: biebl@debian.org

Date: Wed, 25 Jan 2017 13:45:37 UTC

Severity: serious

Tags: buster, sid

Found in version 0.18.debian5

Fixed in version cloud-initramfs-tools/0.18.debian6

Done: Thomas Goirand <zigo@debian.org>

Bug is archived. No further changes may be made.

Toggle useless messages

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to Debian Cloud <debian-cloud@lists.debian.org>:
Bug#852569; Package cloud-initramfs-tools. (Wed, 25 Jan 2017 13:45:40 GMT) (full text, mbox, link).


Acknowledgement sent to biebl@debian.org:
New Bug report received and forwarded. Copy sent to Debian Cloud <debian-cloud@lists.debian.org>. (Wed, 25 Jan 2017 13:45:40 GMT) (full text, mbox, link).


Message #5 received at maintonly@bugs.debian.org (full text, mbox, reply):

From: biebl@debian.org
To: maintonly@bugs.debian.org
Subject: cloud-initramfs-tools: Update udevadm path
Date: Wed, 25 Jan 2017 14:42:36 +0100
Package: cloud-initramfs-tools
Version: 0.18.debian5
Severity: normal
User: pkg-systemd-maintainers@lists.alioth.debian.org
Usertags: udevadm

Hi,

since Jessie, the udevadm binary is available as /bin/udevadm.
To not break existing scripts, which use the old /sbin/udevadm path,
the udev package currently ships a compat symlink which we would like
to drop eventually (in buster or buster+1).

According to codesearch [1] your package cloud-initramfs-tools does hard-code the
udevadm path as /sbin/udevadm.

Please change that to /bin/udevadm.

Thanks for considering.

Michael,
on behalf of the pkg-systemd team.

[1] https://codesearch.debian.net/search?q=%2Fsbin%2Fudevadm




Information forwarded to debian-bugs-dist@lists.debian.org, Debian Cloud <debian-cloud@lists.debian.org>:
Bug#852569; Package cloud-initramfs-tools. (Tue, 20 Jun 2017 12:45:08 GMT) (full text, mbox, link).


Acknowledgement sent to Michael Biebl <biebl@debian.org>:
Extra info received and forwarded to list. Copy sent to Debian Cloud <debian-cloud@lists.debian.org>. (Tue, 20 Jun 2017 12:45:08 GMT) (full text, mbox, link).


Message #10 received at 852569@bugs.debian.org (full text, mbox, reply):

From: Michael Biebl <biebl@debian.org>
To: 852569@bugs.debian.org
Subject: Re: cloud-initramfs-tools: Update udevadm path
Date: Tue, 20 Jun 2017 14:44:42 +0200
On Wed, 25 Jan 2017 14:42:36 +0100 biebl@debian.org wrote:
> Package: cloud-initramfs-tools
> Version: 0.18.debian5
> Severity: normal
> User: pkg-systemd-maintainers@lists.alioth.debian.org
> Usertags: udevadm
> 
> Hi,
> 
> since Jessie, the udevadm binary is available as /bin/udevadm.
> To not break existing scripts, which use the old /sbin/udevadm path,
> the udev package currently ships a compat symlink which we would like
> to drop eventually (in buster or buster+1).
> 
> According to codesearch [1] your package cloud-initramfs-tools does hard-code the
> udevadm path as /sbin/udevadm.
> 
> Please change that to /bin/udevadm.

With stretch being released and buster open for development, it would be
a great opportunity to get this fixed now. So please consider adding
this change in your next upload.

If you are worried about backports: /bin/udevadm is already available in
Debian jessie (oldstable) or Ubuntu trusty (14.04LTS). So it's safe to
use the /bin/udevadm path.

Regards,
Michael

-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?



Information forwarded to debian-bugs-dist@lists.debian.org, Debian Cloud <debian-cloud@lists.debian.org>:
Bug#852569; Package cloud-initramfs-tools. (Tue, 20 Jun 2017 13:09:12 GMT) (full text, mbox, link).


Acknowledgement sent to Michael Biebl <biebl@debian.org>:
Extra info received and forwarded to list. Copy sent to Debian Cloud <debian-cloud@lists.debian.org>. (Tue, 20 Jun 2017 13:09:12 GMT) (full text, mbox, link).


Message #15 received at 852569@bugs.debian.org (full text, mbox, reply):

From: Michael Biebl <biebl@debian.org>
To: 852569@bugs.debian.org
Subject: Re: cloud-initramfs-tools: Update udevadm path
Date: Tue, 20 Jun 2017 14:59:46 +0200
On Wed, 25 Jan 2017 14:42:36 +0100 biebl@debian.org wrote:
> According to codesearch [1] your package cloud-initramfs-tools does hard-code the
> udevadm path as /sbin/udevadm.
> 
> Please change that to /bin/udevadm.

You might also consider not hard-coding the path at all, and simply rely
on PATH being set properly. This should be a safe assumption, especially
since the binary is now in /bin

Regards,
Michael

-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?



Severity set to 'important' from 'normal' Request was from Michael Biebl <biebl@debian.org> to control@bugs.debian.org. (Fri, 30 Jun 2017 09:30:15 GMT) (full text, mbox, link).


Severity set to 'serious' from 'important' Request was from Michael Biebl <biebl@debian.org> to control@bugs.debian.org. (Fri, 08 Sep 2017 18:33:11 GMT) (full text, mbox, link).


Added tag(s) sid and buster. Request was from Andreas Beckmann <anbe@debian.org> to control@bugs.debian.org. (Wed, 13 Sep 2017 10:12:20 GMT) (full text, mbox, link).


Information forwarded to debian-bugs-dist@lists.debian.org, Debian Cloud <debian-cloud@lists.debian.org>:
Bug#852569; Package cloud-initramfs-tools. (Tue, 26 Sep 2017 02:42:05 GMT) (full text, mbox, link).


Acknowledgement sent to Scott Moser <smoser@ubuntu.com>:
Extra info received and forwarded to list. Copy sent to Debian Cloud <debian-cloud@lists.debian.org>. (Tue, 26 Sep 2017 02:42:05 GMT) (full text, mbox, link).


Message #26 received at 852569@bugs.debian.org (full text, mbox, reply):

From: Scott Moser <smoser@ubuntu.com>
To: 852569@bugs.debian.org
Subject: how to use initramfs-tools hook-function's copy-exec
Date: Mon, 25 Sep 2017 22:39:18 -0400 (EDT)
Hi,

In cloud-initramfs-tools, udevadm is referred to by /sbin/udevadm only in
growroot/hooks/growroot with 'copy_exec /sbin/udevadm /sbin'.

It appears that even in most recent version of initramfs-tools only
supports full paths to as copy_exec's first argument.

What is the recommended way to use 'copy_exec' without a full path?

I can make cloud-initramfs-tools do something to the extent of:
  udevadm=$(which udevadm) &&
      copy_exec $udevadm /sbin ||
       { echo "failed copy_exec udevadm"; exit 1 }

But this seems like a generic issue with 'copy_exec' that it requires
full paths to files.

What is the advised solution here?

Thanks,
Scott



Information forwarded to debian-bugs-dist@lists.debian.org, Debian Cloud <debian-cloud@lists.debian.org>:
Bug#852569; Package cloud-initramfs-tools. (Tue, 26 Sep 2017 20:51:04 GMT) (full text, mbox, link).


Acknowledgement sent to Scott Moser <smoser@ubuntu.com>:
Extra info received and forwarded to list. Copy sent to Debian Cloud <debian-cloud@lists.debian.org>. (Tue, 26 Sep 2017 20:51:04 GMT) (full text, mbox, link).


Message #31 received at 852569@bugs.debian.org (full text, mbox, reply):

From: Scott Moser <smoser@ubuntu.com>
To: 852569@bugs.debian.org
Subject: related bug 597580
Date: Tue, 26 Sep 2017 16:50:02 -0400 (EDT)
I'll fix cloud-initramfs-tools to find udevadm in PATH, but
it seems like the right thing to do is to fix:
  https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=597580

And then to make cloud-initramfs-tools just use:
  copy_exec udevadm



Reply sent to Thomas Goirand <zigo@debian.org>:
You have taken responsibility. (Thu, 28 Sep 2017 09:54:04 GMT) (full text, mbox, link).


Notification sent to biebl@debian.org:
Bug acknowledged by developer. (Thu, 28 Sep 2017 09:54:04 GMT) (full text, mbox, link).


Message #36 received at 852569-close@bugs.debian.org (full text, mbox, reply):

From: Thomas Goirand <zigo@debian.org>
To: 852569-close@bugs.debian.org
Subject: Bug#852569: fixed in cloud-initramfs-tools 0.18.debian6
Date: Thu, 28 Sep 2017 09:50:11 +0000
Source: cloud-initramfs-tools
Source-Version: 0.18.debian6

We believe that the bug you reported is fixed in the latest version of
cloud-initramfs-tools, 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 852569@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Thomas Goirand <zigo@debian.org> (supplier of updated cloud-initramfs-tools 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 ftpmaster@ftp-master.debian.org)


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

Format: 1.8
Date: Thu, 28 Sep 2017 11:20:36 +0200
Source: cloud-initramfs-tools
Binary: cloud-initramfs-rescuevol cloud-initramfs-growroot cloud-initramfs-dyn-netconf
Architecture: source all
Version: 0.18.debian6
Distribution: unstable
Urgency: medium
Maintainer: Debian Cloud <debian-cloud@lists.debian.org>
Changed-By: Thomas Goirand <zigo@debian.org>
Description:
 cloud-initramfs-dyn-netconf - write a network interface file in /run for BOOTIF
 cloud-initramfs-growroot - automatically resize the root partition on first boot
 cloud-initramfs-rescuevol - boot off a rescue volume rather than root filesystem
Closes: 852569 872102
Changes:
 cloud-initramfs-tools (0.18.debian6) unstable; urgency=medium
 .
   * Update path of udevadm to /bin (instead of /sbin). (Closes: #852569)
   * Depends on fdisk. (Closes: #872102)
Checksums-Sha1:
 f8b7642d8e74f4b7ad610b877e86852de52c40d9 1931 cloud-initramfs-tools_0.18.debian6.dsc
 9472169f2079dbefb2fd25143c7ddf3a39f2f9d6 32076 cloud-initramfs-tools_0.18.debian6.tar.xz
 8cc27cb4fa658115e0ccf39f5eb674d32300262d 7288 cloud-initramfs-dyn-netconf_0.18.debian6_all.deb
 64772f10d149d09fbb08ffb432c8d237a11343b1 6696 cloud-initramfs-growroot_0.18.debian6_all.deb
 254bd0fd7e55752281cf26589457b4a566f20dcd 7024 cloud-initramfs-rescuevol_0.18.debian6_all.deb
 53d04729a09d9323fd25523254aabd4e8a4c119d 6252 cloud-initramfs-tools_0.18.debian6_amd64.buildinfo
Checksums-Sha256:
 5cf0e03c129dd7612e66782662cca12908b428c8da68dcc7bc7da33367779774 1931 cloud-initramfs-tools_0.18.debian6.dsc
 d6e6cb66ce77409e2e30ce54c51e9cf52c7c685fa1a0778fe0f6b1b3d5cb0b6e 32076 cloud-initramfs-tools_0.18.debian6.tar.xz
 10385449fd87906762b8f7173a81ddfaa77ff7f4b02673666e8a8dcd43af3627 7288 cloud-initramfs-dyn-netconf_0.18.debian6_all.deb
 c2005ea5e21bf55e50347ad2200bb3c11ba031e6ce7bf18ecb599908b9b56651 6696 cloud-initramfs-growroot_0.18.debian6_all.deb
 a23c9fc1313aa6f59fb7d8611ced6be2d30c14a044eff8a56ad2526613764262 7024 cloud-initramfs-rescuevol_0.18.debian6_all.deb
 f7cca98f2afab9daf964981a622bcc037daca5925306030de8cab8a3f2e03e42 6252 cloud-initramfs-tools_0.18.debian6_amd64.buildinfo
Files:
 fd089e70c6a8949f5768427b5c38ce90 1931 admin extra cloud-initramfs-tools_0.18.debian6.dsc
 22bae259dbb4634c0bc53609aa75c525 32076 admin extra cloud-initramfs-tools_0.18.debian6.tar.xz
 bbf50ccc69c5c5e8fd3e74ac225aab70 7288 admin extra cloud-initramfs-dyn-netconf_0.18.debian6_all.deb
 ef4326abb56a6d6e0d49a6717bbea2b7 6696 admin extra cloud-initramfs-growroot_0.18.debian6_all.deb
 6b6e344dfcef1b4c8bbd097dce7cc0ea 7024 admin extra cloud-initramfs-rescuevol_0.18.debian6_all.deb
 1d8b6ad98b6ed683d248381c12714cbc 6252 admin extra cloud-initramfs-tools_0.18.debian6_amd64.buildinfo

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEEoLGp81CJVhMOekJc1BatFaxrQ/4FAlnMwTsACgkQ1BatFaxr
Q/5ljQ/+ItWDpmGM/zPEHB25LBzhW2K/pzFzNot0VN7uG7KCHhibiialSPPhH26h
ztCwYuXu64eDx1rY953e7JUk7iT8BDJzeJ0XXlhLuitoO/9QuemQJ4o4G5eF02iU
MZh3AEuzFAIZn8og6m+cofUP/8bxea+NC0220h2Wz25Wc3yRzNxKLENfA69UvRf2
BqsZ/EFTsI5MG81fNitkjoZMmEB2PgeFFo6sKmN+rrHpdUg+yfXBhkSARe6ebKW0
PvYmtP/QGDzMAWIB6OhHD/6ixS7iiKEgZbvdWOFEiMv5iofE4qLQ2GR8pKSWmvUz
2ptcWFOXKMB4oyf7We9ckzJn1FkXsWQlenqVnD0QZjPEcpT6l2NQBJ5W2jOxEmGb
RkQlBNqtEfT8HvbTHIWfQRgoJWHffODboS+ndDQVsTQFc23IgMKdxcVbOHknh1ks
Vs0Fygo/Wd8Oqj6myDAEGNASd4R5ptXc0V38x6nlEjVGm/XXkhMlfhsR+JK7Ltql
uWIUAnI+Uvolfs0PIVH1DQovaXFBJ3/NzvUECDY/jwj2wW1xlPWz3jvBederdGWB
XezWlPUf2nhFo1xE50pXh4/Fx92vqzW87V54sfv6YcHXuZB3BFCigtMEC39kY/eS
RYRy3mFSAEtKq/8KJ+uPksFlAr5qPwyi0YyZJ7u7dPkJJPi7OnU=
=WIpt
-----END PGP SIGNATURE-----




Bug archived. Request was from Debbugs Internal Request <owner@bugs.debian.org> to internal_control@bugs.debian.org. (Tue, 31 Oct 2017 07:28:18 GMT) (full text, mbox, link).


Send a report that this bug log contains spam.


Debian bug tracking system administrator <owner@bugs.debian.org>. Last modified: Wed Jan 10 14:01:50 2018; Machine Name: beach

Debian Bug tracking system

Debbugs is free software and licensed under the terms of the GNU Public License version 2. The current version can be obtained from https://bugs.debian.org/debbugs-source/.

Copyright © 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson, 2005-2017 Don Armstrong, and many other contributors.