On Wed, Nov 4, 2009 at 15:28, Marco d'Itri <md@xxxxxxxx> wrote: > On Nov 04, Kay Sievers <kay.sievers@xxxxxxxx> wrote: > >> > udev_rules_new() in udev/udev-rules.c unconditionally creates the >> > directory. >> > This is a problem because the function is called also by e.g. udevadm >> > test, and creating /dev/.udev/ when it does not exist is an unacceptable >> > side effect which will break everything else that checks for its >> > existence to know if udev is running. >> > Why is this needed? >> No specific reason, I remember. It could move into the daemon, before >> the inotify watch. > Why create it at all? > (I suspect that this was done to suppress a warning about the missing > directory, but then I'd rather do it properly with a flag...) It's to be able to subscribe to changes in that directory with inotify. We want to create it before we try to watch it, but only the daemon needs to do this. Kay -- To unsubscribe from this list: send the line "unsubscribe linux-hotplug" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
