This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH v2] Implement timestamp'ed output on "make check"

From: Simon Marchi <simon dot marchi at polymtl dot ca>
To: Sergio Durigan Junior <sergiodj at redhat dot com>
Cc: Simon Marchi <simon dot marchi at ericsson dot com>, GDB Patches <gdb-patches at sourceware dot org>, Alan Hayward <Alan dot Hayward at arm dot com>, nd <nd at arm dot com>
Date: Sun, 25 Nov 2018 14:56:32 -0500
Subject: Re: [PATCH v2] Implement timestamp'ed output on "make check"
References: <20181122221240.15354-1-sergiodj@redhat.com> <20181123150256.22584-1-sergiodj@redhat.com> <61b6c448-bae1-2dd9-87a7-5c500be0aa09@ericsson.com> <87wop13z2z.fsf@redhat.com>

On 2018-11-25 11:24, Sergio Durigan Junior wrote:

As for the coding style, according to the wiki [1], we should followPEP8 (which
I think makes sense).  Can you change your script to folow that?
"autopep8 -i print-ts.py" should do it.


Hm, the command doesn't output anything to me, and $? = 0.  Is there
anything in particular that you noticed?

The command modifies the file in-place, so you should have unstagedchanges after running it. In particular, it should remove spaces beforeparentheses.


Simon

Follow-Ups:
- Re: [PATCH v2] Implement timestamp'ed output on "make check"
  - From: Sergio Durigan Junior

References:
- [PATCH] Implement timestamp'ed output on "make check"
  - From: Sergio Durigan Junior
- [PATCH v2] Implement timestamp'ed output on "make check"
  - From: Sergio Durigan Junior
- Re: [PATCH v2] Implement timestamp'ed output on "make check"
  - From: Simon Marchi
- Re: [PATCH v2] Implement timestamp'ed output on "make check"
  - From: Sergio Durigan Junior

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]