From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 582C61B1F7 for ; Fri, 5 Oct 2018 18:00:29 +0200 (CEST) Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 8C17E3168BC4; Fri, 5 Oct 2018 16:00:28 +0000 (UTC) Received: from graphite.drizzt.lan (dhcp189-71.ntdv.lab.eng.bos.redhat.com [10.19.189.71]) by smtp.corp.redhat.com (Postfix) with ESMTP id 316686AEB4; Fri, 5 Oct 2018 16:00:26 +0000 (UTC) Date: Fri, 5 Oct 2018 18:00:25 +0200 From: Timothy Redaelli To: Luca Boccassi Cc: dev@dpdk.org, bruce.richardson@intel.com, christian.ehrhardt@canonical.com, mvarlese@suse.de Message-ID: <20181005180025.12408046@graphite.drizzt.lan> In-Reply-To: <20181002162046.13668-1-bluca@debian.org> References: <20180928175803.12955-1-bluca@debian.org> <20181002162046.13668-1-bluca@debian.org> Organization: Red Hat MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.41]); Fri, 05 Oct 2018 16:00:28 +0000 (UTC) Subject: Re: [dpdk-dev] [PATCH v4 1/2] build: change default PMD installation subdir to dpdk/pmds-XX.YY 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: Fri, 05 Oct 2018 16:00:29 -0000 On Tue, 2 Oct 2018 17:20:45 +0100 Luca Boccassi wrote: > As part of the effort of consolidating the DPDK installation bits and > pieces across distros, set the default directory of lib/ where PMDs get > installed to dpdk/pmds-XX.YY. It's necessary to have a versioned > subdirectory as multiple ABI revisions might be installed at the same > time, so having a fixed name will cause trouble with the autoload > feature. > Small refactor with parsing and saving the major version to a variable, > since it's now used in 3 different places. > > Signed-off-by: Luca Boccassi > --- Acked-by: Timothy Redaelli