From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753489AbbFDM3L (ORCPT ); Thu, 4 Jun 2015 08:29:11 -0400 Received: from cantor2.suse.de ([195.135.220.15]:37713 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752671AbbFDM3F (ORCPT ); Thu, 4 Jun 2015 08:29:05 -0400 Date: Thu, 4 Jun 2015 14:29:03 +0200 From: Petr Mladek To: Joe Perches Cc: Andrew Morton , Andy Whitcroft , linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/3] checkpatch: Improve output with multiple command-line files Message-ID: <20150604122903.GQ3135@pathway.suse.cz> References: <20150604120328.GN3135@pathway.suse.cz> <1433420079.4861.169.camel@perches.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1433420079.4861.169.camel@perches.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu 2015-06-04 05:14:39, Joe Perches wrote: > On Thu, 2015-06-04 at 14:03 +0200, Petr Mladek wrote: > > On Wed 2015-06-03 08:53:38, Joe Perches wrote: > > > If there are multiple patches/files on the command line, > > > use a prefix before the patch/file message output like: > > > -------------- > > > patch/filename > > > -------------- > > > to make the identifying which messages go with which > > > file/patch a bit easier to parse. > [] > > Tested-by: Petr Mladek > [] > > > +NOTE: Whitespace errors detected. > > > + You may wish to use scripts/cleanpatch or scripts/cleanfile > > > +EOM > > > > It would make sense to write this message only once as well. > > I don't think so as it applies to each patch/file separately. IMHO, it advertises scripts/cleanpatch or scripts/cleanfile. I think that we do not need to repeat it for each affected patch. In fact, using scripts/cleanfile is questionable. IMHO, it does not make sense to fix indentation just because it looks better. It makes problems when backporting fixes. But it might make sense to fix it when you do some real change on that line. Best Regards, Petr