Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 465470 - sys-fs/eudev-1_beta4: udev-postmount rcscript is using non-existing `--run` switch for `udevadm info` to get RUNDIR
Summary: sys-fs/eudev-1_beta4: udev-postmount rcscript is using non-existing `--run` s...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: eudev team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-04-11 06:35 UTC by Khayyam
Modified: 2013-04-11 15:59 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Khayyam 2013-04-11 06:35:02 UTC
The udev-postmount init script contained in sys-fs/eudev-1_beta4 uses the following to set ${RUNDIR}:

RUNDIR=$(udevadm info --run)

With that same version of eudev this will return:

# udevadm info --run
info: unrecognized option '--run'

I assume this option was removed (without even a note in systemd-udev's changelog) as its fun to drop command line options and/or break API on a whim.

As things stand the init script does nothing as the test condition for RUNDIR fails due to the above, and so hits '|| return 0'

The solution is probably to hardcode /run/udev but I haven't rebooted in order to test the assumption.

best


Reproducible: Always
Comment 1 Samuli Suominen (RETIRED) gentoo-dev 2013-04-11 06:56:54 UTC
indeed, I can verify --run shouldn't be used anymore, I stumbled upon same issue when I was writing /lib/udev/dev-root-link.sh script, and ended up just hardcoding /run since it's safe to assume it is there
Comment 2 Anthony Basile gentoo-dev 2013-04-11 14:33:57 UTC
(In reply to comment #0)
> The udev-postmount init script contained in sys-fs/eudev-1_beta4 uses the
> following to set ${RUNDIR}:
> 
> RUNDIR=$(udevadm info --run)
> 
> With that same version of eudev this will return:
> 
> # udevadm info --run
> info: unrecognized option '--run'
> 
> I assume this option was removed (without even a note in systemd-udev's
> changelog) as its fun to drop command line options and/or break API on a
> whim.
> 
> As things stand the init script does nothing as the test condition for
> RUNDIR fails due to the above, and so hits '|| return 0'
> 
> The solution is probably to hardcode /run/udev but I haven't rebooted in
> order to test the assumption.
> 
> best
> 
> 
> Reproducible: Always

Thanks!  I know upstream pulls this kind of stuff, but it was our responsibility to catch it.  Sorry.  I'm looking into it.  We may have to hard code.
Comment 3 Ian Stakenvicius (RETIRED) gentoo-dev 2013-04-11 15:59:05 UTC
+*eudev-1_beta4-r1 (11 Apr 2013)
+
+  11 Apr 2013; Ian Stakenvicius <axs@gentoo.org> +eudev-1_beta4-r1.ebuild,
+  -eudev-1_beta3.ebuild, -eudev-1_beta4.ebuild, files/udev-postmount:
+  revbump to fix bug 465470, drop deprecated beta3