Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1503166

Re: [PATCH 2/2] scripts/gdb: fixup some pep8 errors in proc.py

From Peter Griffin <peter.griffin@linaro.org>
Newsgroups linux.kernel
Subject Re: [PATCH 2/2] scripts/gdb: fixup some pep8 errors in proc.py
Date 2016-10-18 18:00 +0200
Message-ID <stEY2-3IP-49@gated-at.bofh.it> (permalink)
References <stEbE-3pl-29@gated-at.bofh.it> <stEbE-3pl-27@gated-at.bofh.it> <stEuZ-3xx-15@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Hi Kieran,

On Tue, 18 Oct 2016, Kieran Bingham wrote:

> Hi Pete,
> 
> On 18/10/16 16:07, Peter Griffin wrote:
> > proc.py:22:1: E302 expected 2 blank lines, found 1
> > proc.py:200:1: E302 expected 2 blank lines, found 1
> > 
> > Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> > ---
> >  scripts/gdb/linux/proc.py | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/scripts/gdb/linux/proc.py b/scripts/gdb/linux/proc.py
> > index f20fcfa..2d6f74e 100644
> > --- a/scripts/gdb/linux/proc.py
> > +++ b/scripts/gdb/linux/proc.py
> > @@ -18,6 +18,7 @@ from linux import tasks
> >  from linux import lists
> >  from struct import *
> >  
> > +
> 
> This was added by patch 1, and can be squashed there.

Doh, I had deliberately done it as a separate commit, as I thought it was
a pre-existing pep8 error.

Will squash in v2.

regards,

Peter.

Back to linux.kernel | Previous | NextPrevious in thread | Find similar | Unroll thread


Thread

[PATCH 2/2] scripts/gdb: fixup some pep8 errors in proc.py Peter Griffin <peter.griffin@linaro.org> - 2016-10-18 17:10 +0200
  Re: [PATCH 2/2] scripts/gdb: fixup some pep8 errors in proc.py Kieran Bingham <kieran@ksquared.org.uk> - 2016-10-18 17:30 +0200
    Re: [PATCH 2/2] scripts/gdb: fixup some pep8 errors in proc.py Peter Griffin <peter.griffin@linaro.org> - 2016-10-18 18:00 +0200

csiph-web