From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bruce Richardson Subject: Re: [PATCH] devtools: rework abi checker script Date: Mon, 4 Sep 2017 17:00:21 +0100 Message-ID: <20170904160020.GA28252@bricha3-MOBL3.ger.corp.intel.com> References: <20170830135154.32092-1-olivier.matz@6wind.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev@dpdk.org, nhorman@tuxdriver.com To: Olivier Matz Return-path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id 9AB067D32 for ; Mon, 4 Sep 2017 18:00:25 +0200 (CEST) Content-Disposition: inline In-Reply-To: <20170830135154.32092-1-olivier.matz@6wind.com> 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 Wed, Aug 30, 2017 at 03:51:54PM +0200, Olivier Matz wrote: > The intiatial version of the script had some limitations: > - cannot work on a non-clean workspace > - environment variables are not documented > - no compilation log in case of failure > - return success even it abi is incompatible > > This patch addresses these issues and rework the code. > > Signed-off-by: Olivier Matz Giving this a spin to test it out. First impression is good, gives nice output of what is happening, though perhaps a little verbose by default. The output finishes though by printing: INFO: HTML reports are in /home/bruce/dpdk-clean/abi-check/compat_reports directory Unfortunately: $ ls -l /home/bruce/dpdk-clean/abi-check/compat_reports /Bruce