From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] devtools: fix checkpatch for filename with space Date: Fri, 20 Jul 2018 22:56:20 +0200 Message-ID: <3233150.zM1BvvQIJO@xps> References: <20180720114103.18211-1-thomas@monjalon.net> <20180720182538.GA21786@neilslaptop.think-freely.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: Neil Horman To: dev@dpdk.org Return-path: Received: from out1-smtp.messagingengine.com (out1-smtp.messagingengine.com [66.111.4.25]) by dpdk.org (Postfix) with ESMTP id 30F1A2B92 for ; Fri, 20 Jul 2018 22:56:26 +0200 (CEST) In-Reply-To: <20180720182538.GA21786@neilslaptop.think-freely.org> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 20/07/2018 20:25, Neil Horman: > On Fri, Jul 20, 2018 at 01:41:03PM +0200, Thomas Monjalon wrote: > > If the patch filename or the temporary file path have a space > > in their name, the script checkpatches.sh does not work. > > The variables for the filenames must be enclosed in quotes > > in order to preserve spaces. > > > > Fixes: 4bec48184e33 ("devtools: add checks for ABI symbol addition") > > > > Signed-off-by: Thomas Monjalon > Acked-by: Neil Horman > > > --- > > > > Strangely, I did a fix for check-symbol-change.sh and I forgot > > to fix checkpatches.sh. > > > > --- Applied