From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH v3] mk: fix compile error and ABI versioning for combined shared library Date: Thu, 3 Dec 2015 11:24:41 +0000 Message-ID: <20151203112441.GA15978@sivlogin002.ir.intel.com> References: <20151203015919.GA23772@sivlogin002.ir.intel.com> <1449108909-29421-1-git-send-email-ferruh.yigit@intel.com> <17459798.JpEqoIGKUu@xps13> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev@dpdk.org To: Thomas Monjalon Return-path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id A1DC737A6 for ; Thu, 3 Dec 2015 12:24:44 +0100 (CET) Content-Disposition: inline In-Reply-To: <17459798.JpEqoIGKUu@xps13> 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, Dec 03, 2015 at 03:22:39AM +0100, Thomas Monjalon wrote: > 2015-12-03 02:15, Ferruh Yigit: > > +ifeq ($(COMBINED_BUILD),1) > > include $(RTE_SDK)/mk/rte.sharelib.mk > > +endif > > I still don't understand what was the issue with this include but it > seems not related to versioning. > Please condider a separate patch with a detailed explanation of the bug. > Indeed this is related to the using versioning macros VERSION_SYMBOL, BIND_DEFAULT_SYMBOL But let me try to fix this in different way > [...] > > +FILES=$(find $RTE_SDK -name "*_version.map") > > No, we can have several build directories in RTE_SDK. > "grep -v" was to remove build output .map files, not searching for _version.map, so it should be OK but let me narrow search path to lib and drivers folders to be safe Thanks, ferruh