On Thu, Nov 27, 2008 at 8:53 AM, Chris Lalancette <span dir="ltr"><<a href="mailto:clalance@redhat.com">clalance@redhat.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Right, both points make sense.  I think the following patch should address it; I<br>
only conditionally set the UDEVADM variable if I find it.  So, for machines<br>
without it, the meat of virStorageBackendWaitForDevices is compiled out.  In<br>
places where I've found it on the build machine, I then do "access" for<br>
executable at runtime, and only if that succeeds do I run it.  Does that seem<br>
correct?  In addition, based on the comment from Guido, I changed it over to use<br>
"udevadm settle" instead of "udevsettle".<br>
<div><div></div></div></blockquote><div><br>Please make it try both: RHEL5 has an ancient udev095 and doesn't have udevadm but it does have udevsettle<br><br></div></div><br>