Eric Sunshine <sunshine@xxxxxxxxxxxxxx> writes: > On Sun, Jun 9, 2013 at 6:22 PM, Célestin Matte > <celestin.matte@xxxxxxxxxx> wrote: >> Changes with v2: >> - Remove patch [02/22] about using the Readonly module >> - Split commit [07/22] into 5 different ones > > This was easier to review after being split. Thanks. > >> - Split commit [14/22] into 2 different ones >> - Patch [17/22] was *not* split: tell me if it is necessary > > [now patch 22/28] > > You, Matthieu, and Junio should decide, but I again found it > time-consuming and onerous to review with all the changes mashed > together. I agree that it would have been better to split the patches in v1, but now that we've already spent time reviewing it, it seems unecessary to spend more time splitting and re-reviewing. I went through the series once more, all my remarks are minor. I'm OK with the series as it is (i.e. perhaps it's time to say "stop the bikeshedding and start coding real stuff" ;-) ). As a reminder: Celestin is in a school project that ends in a week. The goal is both to be productive and to learn stuff. In any case, thanks a lot for your review, Eric. -- Matthieu Moy http://www-verimag.imag.fr/~moy/ -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html