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: <ProtocolError for dpdk.org/dev/patchwork/xmlrpc/: 301 Moved Permanently>

Best Regards,
Gavin

-----Original Message-----
From: Bruce Richardson <redacted>
Sent: Tuesday, June 19, 2018 6:12 PM
To: Gavin Hu <redacted>
Cc: dev@dpdk.org; nd <redacted>; 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:

quoted

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 <redacted>
Reviewed-by: Phil Yang <redacted>
Reviewed-by: Song Zhu <redacted>
---

This is previously acked.