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

Re: [Xen-devel] [patch] libxc whitespace



On Sat, Apr 15, 2006 at 03:30:06PM +0100, John Levon wrote:
> On Sat, Apr 15, 2006 at 10:47:53AM +0300, Muli Ben-Yehuda wrote:
> 
> > > tools/libxc. This patch, a result of running
> > >   perl -p -i -e 's/\s+$/\n/g' tools/libxc/*.[ch]
> > 
> > Could we just run this on the entire Xen and tools trees?
> 
> No, can't we wait and do this a file at a time when other changes are
> made? It is far from critical and it makes merging hell...

I agree it makes merging hell, but hopefully, it will merging hell
once and after that something automatic will enforce it at check-in
time the same way akpm does for -mm, and all will be well with the
world. The strategy you suggest almost always ends up with people
making both changes at once, which makes *reviewing* hell.

Cheers,
Muli
-- 
Muli Ben-Yehuda
http://www.mulix.org | http://mulix.livejournal.com/


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.