Matthew Burgess wrote: > On Tue, 04 Oct 2011 21:45:13 -0700, Bryan Kadzban > <[email protected]> wrote: >> See the attached patch for what I propose we do, at least in the short >> term, or possibly longer as well. >> >> It changes udev_retry to (in addition to using --type=failed) read >> /etc/sysconfig/udev_retry (name TBD, but this works for me?) and pull >> subsystem names from there, then trigger "add" events for those >> subsystems at boot time. > > I really like this patch, Bryan. Thanks for looking into this. > >> Longer term, we can remove the --type=failed invocation entirely, >> although maybe that's better done as part of this change? > > I'd prefer to do that in this change. As you mention in the patch, this won't > work with the current version of Udev in the book, so we may as well get rid.
A review of the patch appears to be OK to me, but I think there are a couple of other issues to consider. First, -type=failed does still work. It's labeled as depreciated and gives a warning message (that is optionally removed with a sed in the build instructions). Having both methods should work for now. Second, the behavior that this adds needs to be documented in the book, either in section 7.4 or a new section after 7.5. Let's also keep in mind here what we are trying to do. This whole thing only occurs if /usr or /var are on separate partitions and most users don't do that. Right now, I think I'd prefer holding this to after 7.0. What I'm waiting for to do the -rc2 release is the availability of packages on kernel.org. -- Bruce -- http://linuxfromscratch.org/mailman/listinfo/lfs-dev FAQ: http://www.linuxfromscratch.org/faq/ Unsubscribe: See the above information page
