Re: Unloading drivers, start-up, shut-down and some clean-ups

[David Brownell <david-b@pacbell.net>]

The first time you sent things around, they weren't in
the form of digestible/reviewable patches ... just an
entire rewrite.  I asked for a more reviewable form,
but the second iteration looked like the same, and
so does this one.

What that does is magnify the time/cost needed to
look at things, and increase the risk of breaking
things when some unexpected interaction doesn't
get noticed in the review, and didn't get fixed.

Rather than complain that I'm not investing that extra
time, perhaps you would consider spending it yourself
to provide a series of MUCH smaller patches, where
each one is reasonably sized and independently
reviewable???

I don't think that's too much to ask.  There's some
good stuff in there, but it's hard to get at it past the
"gratuitous rewrite" type of changes.

- Dave


> I wrote:
> 
> > If nothing else, then somebody at least do the *.agent *.rc script 
> > unification! (please?)
> 
> 
> Um, actually, that 'somebody' is probably David Brownell who might 
> kindly include some of my contributions in CVS, perhaps?
> 
> I have the impression that we are wasting valuable time.



_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel