From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) by dpdk.org (Postfix) with ESMTP id 6E66D1F1C for ; Sat, 27 Oct 2018 02:05:12 +0200 (CEST) Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.nyi.internal (Postfix) with ESMTP id 0BDEF21E09; Fri, 26 Oct 2018 20:05:12 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute1.internal (MEProxy); Fri, 26 Oct 2018 20:05:12 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding:content-type; s=mesmtp; bh=VC45zN4hBJ6v0QAOTe/pcJE56S8d24z4JkkZrA+hhu8=; b=CsmxrCZM3hqA AtSofvU1D9C32LN2krQMkUjzYqm7ZEjpgbt/oFnx3GBkJbD4MrCB2gA7xaKzlBr9 D5q36HCK9KtT6oqwZ/nEoqvMaTM8fdhaJ9MLRuGVs7KrCrgk+CPE7C+ZTpVoyfQM y9RwIzn/Z6lT6PHaN6GWXkNdfYx2aQs= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm1; bh=VC45zN4hBJ6v0QAOTe/pcJE56S8d24z4JkkZrA+hh u8=; b=ys6MOpqtWbLHF8ndmIEXz70dbzIoZGAloCZp/uStUqO11Xmy0ebQ4Jsz+ 1xtJtlrgvGlhEj/dppCAf5zeK3H71TytYBNjKNCj1YifhwrYsVcG3vO6vJpuFz8k y0iLlrk3L4O7j2XJnFo1oPWii48WfTra7ivVGYOc31cy/pBnPmiu2UGvcv3HN+tx 3FfQ2YXyxlCGhrgYwyxKCpy6JfWvtz9VqccICO9//zUF16GEOUzfCLGew/OxYT/f /wiP64fuBUy38g8pwNTHd5o7XVePm7icBzSei/KncNzWpQ/3HrrcEnTZJOp5fd4z /yCIGWF9HVllnQCwryCq4aIn0oyPA== X-ME-Sender: X-ME-Proxy: Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id AB53FE4074; Fri, 26 Oct 2018 20:05:10 -0400 (EDT) From: Thomas Monjalon To: Ferruh Yigit Cc: dev@dpdk.org Date: Sat, 27 Oct 2018 02:05:16 +0200 Message-ID: <1555566.RdUPOnmJgc@xps> In-Reply-To: References: <20181026184248.78908-1-ferruh.yigit@intel.com> <2054335.yDqyzkpi5h@xps> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [PATCH v2] buildtools: fix build for some mktemp X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 27 Oct 2018 00:05:12 -0000 27/10/2018 01:53, Ferruh Yigit: > On 10/27/2018 12:47 AM, Thomas Monjalon wrote: > > 27/10/2018 02:38, Ferruh Yigit: > >> build error: > >> == Build drivers/net/tap > >> mktemp: cannot create temp file /tmp/dpdk.auto-config-h.sh.XXX.c: > >> Invalid argument > >> .../buildtools/auto-config-h.sh: line 86: : No such file or directory > >> .../drivers/net/tap/Makefile:55: recipe for target > >> 'tap_autoconf.h.new' failed > >> > >> Above error observed on Wind River Linux 8.0 > >> > >> `mktemp` command in that system has a restrictions to have X in > >> the template at the end and at least six of them. > >> > >> Complied to mktemp requirements and add -xc flag to compiler to say > >> `temp` file is a C file > >> > >> Fixes: ff37ca5d3773 ("devtools: use a common prefix for temporary files") > >> > >> Reported-by: Shuai Zhu > >> Signed-off-by: Ferruh Yigit > >> --- > >> v2: > >> * keep using mktemp, add -xc to compiler > >> --- > >> buildtools/auto-config-h.sh | 4 ++-- > >> 1 file changed, 2 insertions(+), 2 deletions(-) > >> > >> --- a/buildtools/auto-config-h.sh > >> +++ b/buildtools/auto-config-h.sh > >> -temp=$(mktemp -t dpdk.${0##*/}.XXX.c) > >> +temp=$(mktemp -t dpdk.${0##*/}.c.XXXXXX) > > > > It looks OK for this script. > > > > Should we do the same kind of change for devtools/check-includes.sh? > > and devtools/cocci.sh? > > Not sure, this script is part of buildtool and required for build, so needs to > be compatible for various systems. Above are devtools and can use new mktemp. You mean nobody will use Wind River Linux 8.0 for developing? You are probably right :)