From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bruce Richardson Subject: Re: [PATCH v11 4/6] devtools: fix the missing ninja command error Date: Tue, 19 Jun 2018 13:30:09 +0100 Message-ID: <20180619123009.GA20032@bricha3-MOBL.ger.corp.intel.com> References: <20180615102143.12778-1-gavin.hu@arm.com> <20180619013746.13268-1-gavin.hu@arm.com> <20180619013746.13268-5-gavin.hu@arm.com> <20180619101220.GB23348@bricha3-MOBL.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: "dev@dpdk.org" , nd , "stable@dpdk.org" To: Gavin Hu Return-path: Content-Disposition: inline In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Tue, Jun 19, 2018 at 10:41:06AM +0000, Gavin Hu wrote: > Hi Bruce, > > Thanks for your review, I submitted v12 patch to address all your new comments. > > BTW, do you know why pwclient can NOT work successfully today? > It worked fine at my side previously, any changes at the server side? The error is below: > xmlrpc.client.ProtocolError: > > Best Regards, > Gavin Paths on the server have changed, so you need to update your .pwclientrc file to point to the new path. Patchwork is now at "patches.dpdk.org" > > > -----Original Message----- > > From: Bruce Richardson > > Sent: Tuesday, June 19, 2018 6:12 PM > > To: Gavin Hu > > Cc: dev@dpdk.org; nd ; stable@dpdk.org > > Subject: Re: [dpdk-dev] [PATCH v11 4/6] devtools: fix the missing ninja > > command error > > > > On Tue, Jun 19, 2018 at 09:37:44AM +0800, Gavin Hu wrote: > > > On some linux distributions, eg: CentOS, the ninja executable has a > > > different name: ninja-build, this patch is to check and adapt to it > > > accordingly. > > > > > > ./devtools/test-meson-builds.sh: line 24: ninja: command not found > > > > > > Fixes: a55277a788 ("devtools: add test script for meson builds") > > > Cc: stable@dpdk.org > > > > > > Signed-off-by: Gavin Hu > > > Reviewed-by: Phil Yang > > > Reviewed-by: Song Zhu > > > --- > > This is previously acked.