From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergio Gonzalez Monroy Subject: Re: [PATCH 0/4] Fix build issues with CONFIG_RTE_BUILD_COMBINE_LIBS=y Date: Fri, 3 Oct 2014 09:10:19 +0100 Message-ID: <20141003081019.GA28988@sivswdev02.ir.intel.com> References: <1412265386-26291-1-git-send-email-sergio.gonzalez.monroy@intel.com> <20141002172634.GE4900@hmsreliant.think-freely.org> <20141002200420.GB29590@mhcomputing.net> <2041475.WSUx3LgNfR@xps13> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev-VfR2kkLFssw@public.gmane.org To: Thomas Monjalon Return-path: Content-Disposition: inline In-Reply-To: <2041475.WSUx3LgNfR@xps13> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" On Fri, Oct 03, 2014 at 09:15:20AM +0200, Thomas Monjalon wrote: > 2014-10-02 13:04, Matthew Hall: > > On Thu, Oct 02, 2014 at 01:26:34PM -0400, Neil Horman wrote: > > > 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) > > > > Not clear if you wrote "single shared library" on purpose instead of "single > > static library". But for me the objective of COMBINE_LIBS usage would be > > getting a "single static library" for my app, which just works, and eliminates > > need of start-group, end-group, weird library ordering issues, etc. I'm not > > interested personally in a "shared library" because it'd run slower. > > > > Personally my preference would be to do both the single libs and multiple libs > > in static format by default. Disk space is cheap, let's maximize user freedom > > and flexibility. But shared lib, since it performs less well, should be > > discouraged by default, although allowed if needed... some people prefer it > > because it's easier to patch security vulns if you can replace a buggy library > > for all the code on a system. > > We need to simplify build options. So I'm fine to remove COMBINE_LIBS option > to always enable it. > About making only one single static library, I think it's a good idea if > it brings a real code simplification. > > So the conclusion is to nack this patchset in favor of above changes. > Sergio, comments? > Frankly I did not think of users linking against single and combine lib for different apps. I think If the goal is to simplify code then we should just provide one build option, either single or combine. Personally, I do not have a preference. So just to be clear, we would remove COMBINE_LIBS to always make a single combine lib or to create both single and combine? For the later option, would we be linking apps against single or combine libraries? Sergio > -- > Thomas