Merge lp:~javier.collado/checkbox/bug597305 into lp:checkbox

[Javier Collado]

/sbin directory added to path so that udevadm command can be found even if /sbin isn't in the PATH environment variable.

[Marc Tardif (cr3)]

Under which circumstance does this happen?

review: Needs Information

[Javier Collado (javier.collado)]

I have seen a report in which this happened as explained in bug597305. However, I don't really know why PATH doesn't contain /sbin directory. Anyway, I believe this isn't mandatory and checkbox shouldn't rely on it. Do you agree?

[Javier Collado (javier.collado)]

@Marc,

Any update on this?

[Marc Tardif (cr3)]

I'd like to understand under what circumstances /sbin is not in the PATH variable before adding it to the environment.

review: Needs Information

[Javier Collado (javier.collado)]

Since I'm not able to reproduce the problem I'll reject the merge proposal and mark the bug as 'Won't fix'.

If I find the same problem in the future I'll try to gather the required information.

review: Disapprove

Unmerged revisions

823. By Javier Collado on 2010-06-22

Adding /sbin path to udevadm command

=== modified file 'scripts/udev_resource'
--- scripts/udev_resource	2010-04-22 21:12:51 +0000
+++ scripts/udev_resource	2010-06-22 15:32:29 +0000
@@ -34,7 +34,7 @@
 
 
 # Command to retrieve udev information.
-COMMAND = "udevadm info --export-db"
+COMMAND = "/sbin/udevadm info --export-db"
 
 
 class DeviceResource(object):