From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bruce Richardson Subject: Re: [RFC 4/4] app/test: support resources archived by tar Date: Fri, 6 May 2016 10:02:07 +0100 Message-ID: <20160506090206.GA14368@bricha3-MOBL3> References: <1461935496-20367-1-git-send-email-viktorin@rehivetech.com> <1461935496-20367-5-git-send-email-viktorin@rehivetech.com> <20160505133300.GA15140@bricha3-MOBL3> <20160505190540.5e4c8422@pcviktorin.fit.vutbr.cz> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev@dpdk.org, Thomas Monjalon , David Marchand To: Jan Viktorin Return-path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 0404F47D0 for ; Fri, 6 May 2016 11:02:17 +0200 (CEST) Content-Disposition: inline In-Reply-To: <20160505190540.5e4c8422@pcviktorin.fit.vutbr.cz> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Thu, May 05, 2016 at 07:05:40PM +0200, Jan Viktorin wrote: > On Thu, 5 May 2016 14:33:01 +0100 > Bruce Richardson wrote: > > > On Fri, Apr 29, 2016 at 03:11:36PM +0200, Jan Viktorin wrote: > > > When needing a more complex resource (a file hierarchy), packing every single > > > file as a single resource would be very ineffective. For that purpose, it is > > > possible to pack the files into a tar archive, extract it before test from the > > > resource and finally clean up all the created files. > > > > > > This patch introduces functions resource_untar and resource_rm_by_tar to > > > perform those tasks. An example of using those functions is included as a test. > > > > > > Signed-off-by: Jan Viktorin > > We might want to make this configurable on/off at build time, since it adds a new > > dependency for building dpdk - libarchive-devel. > > This is true. Unfortunately... > > Is it a problem to introduce such dependency into the test app? > Would it be help to use another similar (more spread) library (zlib)? > Not sure it would make much difference, to be honest. It all depends on what -devel packages are installed by default on most distros when you install the development tools sets. I'm not concerned too much either way, but if we add in a new dependency like this, we should also take the opportunity to turn on pcap PMD by default too and make libpcap-devel a dependency. /Bruce