Re: Patch reviews

2011-06-09 Thread Ralf Wildenhues
Hello guys, On Tue, Jun 07, 2011 at 09:25:33AM +0200, Peter Rosin wrote: > Now that Ralf is doing something else for a while, Thanks for the roses. After moving twice within 5 weeks (and generally having far too little time), things should gradually improve as soon as I have a landline internet

Re: Patch reviews

2011-06-07 Thread Peter Rosin
>> You need to read through your own patches more carefully. Slow down. >> What's the rush? > >> >> Especially when there is virtually no review process. >> You are also not inviting review when you mostly push FYI patches >> where the review is an afterthought at best.

Patch reviews (was: Re: [FYI] {testsuite-work} tests: use `$SHELL' to run the shell scripts from `lib/')

2011-06-05 Thread Stefano Lattarini
rush? > > Especially when there is virtually no review process. > You are also not inviting review when you mostly push FYI patches > where the review is an afterthought at best. > I was doing so just because Ralf had made clear that, due to personal reasons, he's unfortunately