From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.tuxdriver.com (charlotte.tuxdriver.com [70.61.120.58]) by dpdk.org (Postfix) with ESMTP id AD01E7DF0 for ; Thu, 2 Oct 2014 19:19:52 +0200 (CEST) Received: from [2001:470:8:a08:9833:6894:f2b2:43a] (helo=localhost) by smtp.tuxdriver.com with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.63) (envelope-from ) id 1XZk9C-0002lM-Or; Thu, 02 Oct 2014 13:26:41 -0400 Date: Thu, 2 Oct 2014 13:26:34 -0400 From: Neil Horman To: Sergio Gonzalez Monroy Message-ID: <20141002172634.GE4900@hmsreliant.think-freely.org> References: <1412265386-26291-1-git-send-email-sergio.gonzalez.monroy@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1412265386-26291-1-git-send-email-sergio.gonzalez.monroy@intel.com> User-Agent: Mutt/1.5.23 (2014-03-12) X-Spam-Score: -2.9 (--) X-Spam-Status: No Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH 0/4] Fix build issues with CONFIG_RTE_BUILD_COMBINE_LIBS=y X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 02 Oct 2014 17:19:52 -0000 On Thu, Oct 02, 2014 at 04:56:22PM +0100, Sergio Gonzalez Monroy wrote: > When building DPDK with CONFIG_RTE_BUILD_COMBINE_LIBS=y, the result is not > the expected behavior. > > - It does link the combine library using LD instead of CC which results > in application linking errors. > > - It creates both individual libraries and combine library, then linking > applications against all of them. > > This patch set aims to fix those issues. > > The last patch 'cleanup', in my opinion, simplifies and removes duplication of > rules. > It is not required for fixing the issues mentioned above. > > Sergio Gonzalez Monroy (4): > Link combined shared library using CC > Do not generate individual libs when configured with RTE_BUILD_COMBINE_LIBS=y > Link apps only against combined lib or individual libs, not both > Cleanup > > mk/rte.app.mk | 13 +++++--- > mk/rte.lib.mk | 90 +++++++++++++----------------------------------------- > mk/rte.sharelib.mk | 47 ++++++++++++++-------------- > 3 files changed, 54 insertions(+), 96 deletions(-) > > Signed-off-by: Sergio Gonzalez Monroy > > -- > 1.9.3 > > Just out of curiosity, whats the impetus behind a single shared library here? Is it just to ease application linking operations? If so, it almost seems to me that we should abandon the individual linking method and just use this as the default output (and do simmilarly for the static linking build) Neil