[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 00/17] linux-user patches in maemo

From: Aurelien Jarno
Subject: Re: [Qemu-devel] [PATCH 00/17] linux-user patches in maemo
Date: Wed, 1 Apr 2009 14:07:30 +0200
User-agent: Mutt/1.5.18 (2008-05-17)

On Wed, Apr 01, 2009 at 12:31:17PM +0300, Riku Voipio wrote:
> On Tue, Mar 31, 2009 at 10:31:42PM +0000, Paul Brook wrote:
> > > This some of the more cleaner linux-user patches maemo version
> > > of qemu carries. The idea was to setup a "linux-user-for-upstream"
> > > branch to git.maemo.org, but unfortunately the server is refusink
> > > my git pushes ATM.
> > These patches don't seem particularly coherent.
> > Some of them actually depend on preceding patches, others implement 
> > completely 
> > independent features.
> No problem, I can split these to smaller series.
> > In some places you modify code, only to change that code again in a later 
> > patch. For example patch #7 appears to be purely a fix for bugs introduced 
> > by 
> > patch #5
> These specific patches are unmodified patches submitted by Kirill in October
> last year, without anyone complaining.. I'll squash patches together where
> newer ones changes code of older patches.

The one from October are actually a resent of an earlier series. I
haven't look at the one you selected, but from what I remember they have
a lot of unrelated indentation change, making them difficult to review.

Aurelien Jarno                          GPG: 1024D/F1BCDB73
address@hidden                 http://www.aurel32.net

reply via email to

[Prev in Thread] Current Thread [Next in Thread]