From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 3D233A046B for ; Fri, 28 Jun 2019 21:40:11 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id C54784C93; Fri, 28 Jun 2019 21:40:09 +0200 (CEST) Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by dpdk.org (Postfix) with ESMTP id 9240AF64 for ; Fri, 28 Jun 2019 21:40:07 +0200 (CEST) X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from orsmga007.jf.intel.com ([10.7.209.58]) by orsmga106.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 28 Jun 2019 12:40:06 -0700 X-IronPort-AV: E=Sophos;i="5.63,428,1557212400"; d="scan'208";a="153462815" Received: from bricha3-mobl.ger.corp.intel.com ([10.252.21.39]) by orsmga007-auth.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 28 Jun 2019 12:40:05 -0700 Date: Fri, 28 Jun 2019 20:40:02 +0100 From: Bruce Richardson To: Stephen Hemminger Cc: Ferruh Yigit , Thomas Monjalon , dev@dpdk.org Message-ID: <20190628194001.GC366@bricha3-MOBL.ger.corp.intel.com> References: <20190625170107.87583-1-ferruh.yigit@intel.com> <20190628092747.2c77eed1@hermes.lan> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190628092747.2c77eed1@hermes.lan> User-Agent: Mutt/1.11.4 (2019-03-13) Subject: Re: [dpdk-dev] [PATCH] devtools: return error on failure 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Fri, Jun 28, 2019 at 09:27:47AM -0700, Stephen Hemminger wrote: > On Tue, 25 Jun 2019 18:01:06 +0100 > Ferruh Yigit wrote: > > > Currently script is always returning success, returning failure on error > > helps on using script on automation tools. > > > > Signed-off-by: Ferruh Yigit > > Looks good, but maybe this script is getting complex enough that > it should be python? > > > Acked-by: Stephen Hemminger +1