Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760815AbYAKJXo (ORCPT ); Fri, 11 Jan 2008 04:23:44 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1757856AbYAKJX3 (ORCPT ); Fri, 11 Jan 2008 04:23:29 -0500 Received: from ns.firmix.at ([62.141.48.66]:4377 "EHLO ns.firmix.at" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756699AbYAKJX2 (ORCPT ); Fri, 11 Jan 2008 04:23:28 -0500 Subject: Re: [PATCH] checkpatch.pl: allow piping From: Bernd Petrovitsch To: Jiri Slaby Cc: Daniel Walker , apw@shadowen.org, linux-kernel@vger.kernel.org, rdunlap@xenotime.net, jschopp@austin.ibm.com In-Reply-To: <47873505.7040008@gmail.com> References: <20080111041120.085610726@mvista.com> <47872E35.3070705@gmail.com> <1200043035.29897.62.camel@imap.mvista.com> <47873505.7040008@gmail.com> Content-Type: text/plain Organization: Firmix Software GmbH Date: Fri, 11 Jan 2008 10:23:19 +0100 Message-Id: <1200043399.3905.9.camel@gimli.at.home> Mime-Version: 1.0 X-Mailer: Evolution 2.10.3 (2.10.3-4.fc7) Content-Transfer-Encoding: 7bit X-Firmix-Scanned-By: MIMEDefang 2.56 on ns.firmix.at X-Firmix-Spam-Score: -2.334 () AWL,BAYES_00,FORGED_RCVD_HELO,SPF_HELO_PASS,SPF_PASS X-Firmix-Spam-Status: No, hits=-2.334 required=5 X-Spam-Score: -2.334 () AWL,BAYES_00,FORGED_RCVD_HELO,SPF_HELO_PASS,SPF_PASS X-Firmix-Envelope-From: X-Firmix-Envelope-To: Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1383 Lines: 33 On Fre, 2008-01-11 at 10:21 +0100, Jiri Slaby wrote: > On 01/11/2008 10:17 AM, Daniel Walker wrote: > > On Fri, 2008-01-11 at 09:52 +0100, Jiri Slaby wrote: > >> On 01/11/2008 05:10 AM, Daniel Walker wrote: > >>> A little feature addition to allow checkpatch.pl to check patches piped > >>> into it, in addition to specific file arguments. > >> You can still add - as an argument to check stdin. In which way is this better? > > > > There's was no reason to limit the arguments .. > > > > I was using it to do something like the following , > > > > git show 9914cad54c79d0b89b1f066c0894f00e1344131c | ./scripts/checkpatch.pl > > Ok, and if you add a - there, it should have the same effect, but for free, > doesn't it: > git show 9914cad54c79d0b89b1f066c0894f00e1344131c | ./scripts/checkpatch.pl - JftSoC: git show 9914cad54c79d0b89b1f066c0894f00e1344131c | ./scripts/checkpatch.pl /dev/stdin (and there a a few others) should also do the trick. Bernd -- Firmix Software GmbH http://www.firmix.at/ mobil: +43 664 4416156 fax: +43 1 7890849-55 Embedded Linux Development and Services -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/