From: "Gonzalez Monroy, Sergio" <sergio.gonzalez.monroy-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
To: Stephen Hemminger
<stephen-OTpzqLSitTUnbdJkjeBofR2eb7JE58TQ@public.gmane.org>
Cc: dev-VfR2kkLFssw@public.gmane.org
Subject: Re: [PATCH v3 1/5] mk: remove combined library and related options
Date: Thu, 09 Apr 2015 09:33:25 +0100 [thread overview]
Message-ID: <55263955.1070707@intel.com> (raw)
In-Reply-To: <20150408112619.14596b65@urahara>
On 08/04/2015 19:26, Stephen Hemminger wrote:
> On Wed, 8 Apr 2015 16:07:21 +0100
> Sergio Gonzalez Monroy <sergio.gonzalez.monroy-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> wrote:
>
>> Currently, the target/rules to build combined libraries is different
>> than the one to build individual libraries.
>>
>> By removing the combined library option as a build configuration option
>> we simplify the build pocess by having a single point for linking/archiving
>> libraries in DPDK.
>>
>> This patch removes CONFIG_RTE_BUILD_COMBINE_LIB build config option and
>> removes the makefiles associated with building a combined library.
>>
>> The CONFIG_RTE_LIBNAME config option is kept as it will be use to
>> always generate a linker script that acts as a single combined library.
>>
>> Signed-off-by: Sergio Gonzalez Monroy <sergio.gonzalez.monroy-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
> No. We use combined library and it greatly simplfies the application
> linking process.
>
After all the opposition this patch had in v2, I did explain the current
issues
(see http://dpdk.org/ml/archives/dev/2015-March/015366.html ) and this
was the agreed solution.
As I mention in the cover letter (also see patch 2/5), building DPDK
(after applying this patch series) will always generate a very simple
linker script that behaves as a combined library.
I encourage you to apply this patch series and try to build your app
(which links against combined lib).
Your app should build without problem unless I messed up somewhere and
it needs fixing.
Sergio
next prev parent reply other threads:[~2015-04-09 8:33 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-08 15:07 [PATCH v3 0/5] Enhance build process Sergio Gonzalez Monroy
[not found] ` <1428505645-5578-1-git-send-email-sergio.gonzalez.monroy-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2015-04-08 15:07 ` [PATCH v3 1/5] mk: remove combined library and related options Sergio Gonzalez Monroy
[not found] ` <1428505645-5578-2-git-send-email-sergio.gonzalez.monroy-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2015-04-08 18:26 ` Stephen Hemminger
2015-04-09 8:33 ` Gonzalez Monroy, Sergio [this message]
[not found] ` <55263955.1070707-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2015-04-09 9:06 ` Avi Kivity
[not found] ` <55264127.2020604-RmZWMc9puTNJc61us3aD9laTQe2KTcn/@public.gmane.org>
2015-04-09 9:42 ` Gonzalez Monroy, Sergio
2015-04-09 11:19 ` Neil Horman
[not found] ` <20150409111943.GA26201-B26myB8xz7F8NnZeBjwnZQMhkBWG/bsMQH7oEaQurus@public.gmane.org>
2015-04-09 13:33 ` Thomas Monjalon
2015-04-09 13:47 ` Gonzalez Monroy, Sergio
2015-04-09 17:00 ` Avi Kivity
[not found] ` <5526B02A.70700-RmZWMc9puTNJc61us3aD9laTQe2KTcn/@public.gmane.org>
2015-04-09 20:34 ` Neil Horman
[not found] ` <20150409203409.GB29807-B26myB8xz7F8NnZeBjwnZQMhkBWG/bsMQH7oEaQurus@public.gmane.org>
2015-04-13 9:52 ` Gonzalez Monroy, Sergio
[not found] ` <552B91E5.9070803-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2015-04-13 11:04 ` Thomas Monjalon
2015-04-13 11:23 ` Neil Horman
2015-04-08 15:07 ` [PATCH v3 2/5] mk: always generate combined lib linker script Sergio Gonzalez Monroy
2015-04-08 15:07 ` [PATCH v3 3/5] lib: set LDLIBS for each library Sergio Gonzalez Monroy
2015-04-08 15:07 ` [PATCH v3 4/5] mk: use LDLIBS when linking shared libraries Sergio Gonzalez Monroy
2015-04-08 15:07 ` [PATCH v3 5/5] mk: update LDLIBS for app building Sergio Gonzalez Monroy
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=55263955.1070707@intel.com \
--to=sergio.gonzalez.monroy-ral2jqcrhueavxtiumwx3w@public.gmane.org \
--cc=dev-VfR2kkLFssw@public.gmane.org \
--cc=stephen-OTpzqLSitTUnbdJkjeBofR2eb7JE58TQ@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.