Re: [Qemu-devel] [Qemu-trivial] [PATCH trivial] make: fix clean rule by removing build file in qom/

2012-04-03 Thread Stefan Hajnoczi
On Tue, Apr 03, 2012 at 11:06:45AM +0100, Anthony PERARD wrote:
> On Tue, Apr 3, 2012 at 09:56, Stefan Hajnoczi  wrote:
> >
> >> tg: (a3b6181..) fix/make-clean (depends on: master)
> >
> > What's this?  I'm curious.  First I thought it was a signature joke that
> > I didn't get but I guess it's for some kind of commit tracking system?
> 
> No, it is not. To prepare a patch (or patch set), I use a tool called topgit.
> 
> http://repo.or.cz/w/topgit.git
> 
> It creates one git branch per patch, and you can add as many
> dependence branch as you want. When you want to "rebase" a patch,
> topgit will merge with the depends.

Thanks for sharing!

Stefan



Re: [Qemu-devel] [Qemu-trivial] [PATCH trivial] make: fix clean rule by removing build file in qom/

2012-04-03 Thread Anthony PERARD
On Tue, Apr 3, 2012 at 09:56, Stefan Hajnoczi  wrote:
>
>> tg: (a3b6181..) fix/make-clean (depends on: master)
>
> What's this?  I'm curious.  First I thought it was a signature joke that
> I didn't get but I guess it's for some kind of commit tracking system?

No, it is not. To prepare a patch (or patch set), I use a tool called topgit.

http://repo.or.cz/w/topgit.git

It creates one git branch per patch, and you can add as many
dependence branch as you want. When you want to "rebase" a patch,
topgit will merge with the depends.

-- 
Anthony PERARD



Re: [Qemu-devel] [Qemu-trivial] [PATCH trivial] make: fix clean rule by removing build file in qom/

2012-04-03 Thread Stefan Hajnoczi
On Mon, Apr 02, 2012 at 04:10:39PM +0100, Anthony PERARD wrote:
> Make clean does not clean the 'qom' directory, leaving *.o and *.d files. This
> patch fixes this.
> 
> Signed-off-by: Anthony PERARD 
> 
> ---
>  Makefile |1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)

Thanks, applied to the trivial patches tree:
https://github.com/stefanha/qemu/commits/trivial-patches

> tg: (a3b6181..) fix/make-clean (depends on: master)

What's this?  I'm curious.  First I thought it was a signature joke that
I didn't get but I guess it's for some kind of commit tracking system?

Stefan