From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id 86C2323D for ; Mon, 18 Dec 2017 14:50:42 +0100 (CET) X-Amp-Result: UNSCANNABLE X-Amp-File-Uploaded: False Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by orsmga105.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 18 Dec 2017 05:50:41 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.45,422,1508828400"; d="scan'208";a="2699978" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.221.106]) by fmsmga002.fm.intel.com with SMTP; 18 Dec 2017 05:50:39 -0800 Received: by (sSMTP sendmail emulation); Mon, 18 Dec 2017 13:50:38 +0000 Date: Mon, 18 Dec 2017 13:50:38 +0000 From: Bruce Richardson To: Harry van Haaren Cc: dev@dpdk.org Message-ID: <20171218135037.GA15444@bricha3-MOBL3.ger.corp.intel.com> References: <1513598038-148115-1-git-send-email-harry.van.haaren@intel.com> <1513598038-148115-2-git-send-email-harry.van.haaren@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1513598038-148115-2-git-send-email-harry.van.haaren@intel.com> Organization: Intel Research and Development Ireland Ltd. User-Agent: Mutt/1.9.1 (2017-09-22) Subject: Re: [dpdk-dev] [PATCH 1/2] test: use env variable to run test if set 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: Mon, 18 Dec 2017 13:50:42 -0000 On Mon, Dec 18, 2017 at 11:53:57AM +0000, Harry van Haaren wrote: > This commit paves the way for the meson tests in the next > patch. With this patch the test binary checks the DPDK_TEST > environment variable and if set, the contents of the var > are inserted on the test app command line, and run. > > This allows testing of various different unit tests without > manual interaction with the RTE>> test prompt, instead automating > it using the DPDK_TEST environment variable. > > If the DPDK_TEST env variable is not set, or has zero lenght, > the test app behaves as normal. > > Signed-off-by: Harry van Haaren > --- This is very useful, even without the meson test part added in the next patch. Probably a matter of personal taste, but I think I would prefer to see the tests to be run specified as args on the cmdline rather than passed through the environment. I'd tend to view environment variables as things to use for long-lasting values, rather than parameters you might want to change between each run. Using cmdline args would also save you having to split commands, since you'd get the list pre-split in argv. /Bruce