Re: [Qemu-devel] [PULL 44/50] scripts: let checkpatch.pl process an entire GIT branch

2017-10-04 Thread Alex Williamson
On Wed, 4 Oct 2017 16:11:07 +0200 Paolo Bonzini wrote: > On 04/10/2017 15:17, Alex Williamson wrote: > > Yes, it works with the new --patch option. I use stgit which does > > not add a filename extension to the patch name on export, nor have I > > ever known patch files to have any accepted stan

Re: [Qemu-devel] [PULL 44/50] scripts: let checkpatch.pl process an entire GIT branch

2017-10-04 Thread Daniel P. Berrange
On Wed, Oct 04, 2017 at 04:11:07PM +0200, Paolo Bonzini wrote: > On 04/10/2017 15:17, Alex Williamson wrote: > > Yes, it works with the new --patch option. I use stgit which does > > not add a filename extension to the patch name on export, nor have I > > ever known patch files to have any accepte

Re: [Qemu-devel] [PULL 44/50] scripts: let checkpatch.pl process an entire GIT branch

2017-10-04 Thread Paolo Bonzini
On 04/10/2017 15:17, Alex Williamson wrote: > Yes, it works with the new --patch option. I use stgit which does > not add a filename extension to the patch name on export, nor have I > ever known patch files to have any accepted standard file extension. > checkpatch.pl has always assumed it's oper

Re: [Qemu-devel] [PULL 44/50] scripts: let checkpatch.pl process an entire GIT branch

2017-10-04 Thread Alex Williamson
On Wed, 4 Oct 2017 09:33:29 +0100 "Daniel P. Berrange" wrote: > On Tue, Oct 03, 2017 at 04:07:06PM -0600, Alex Williamson wrote: > > On Tue, 19 Sep 2017 14:29:33 +0200 > > Paolo Bonzini wrote: > > > > > From: "Daniel P. Berrange" > > > > > > Currently before submitting a series, devs should

Re: [Qemu-devel] [PULL 44/50] scripts: let checkpatch.pl process an entire GIT branch

2017-10-04 Thread Daniel P. Berrange
On Tue, Oct 03, 2017 at 04:07:06PM -0600, Alex Williamson wrote: > On Tue, 19 Sep 2017 14:29:33 +0200 > Paolo Bonzini wrote: > > > From: "Daniel P. Berrange" > > > > Currently before submitting a series, devs should run checkpatch.pl > > across each patch to be submitted. This can be automated

Re: [Qemu-devel] [PULL 44/50] scripts: let checkpatch.pl process an entire GIT branch

2017-10-03 Thread Alex Williamson
On Tue, 19 Sep 2017 14:29:33 +0200 Paolo Bonzini wrote: > From: "Daniel P. Berrange" > > Currently before submitting a series, devs should run checkpatch.pl > across each patch to be submitted. This can be automated using a > command such as: > > git rebase -i master -x 'git show | ./scripts

[Qemu-devel] [PULL 44/50] scripts: let checkpatch.pl process an entire GIT branch

2017-09-19 Thread Paolo Bonzini
From: "Daniel P. Berrange" Currently before submitting a series, devs should run checkpatch.pl across each patch to be submitted. This can be automated using a command such as: git rebase -i master -x 'git show | ./scripts/checkpatch.pl -' This is rather long winded to type, so this patch int