luca.boccassi at gmail.com
Wed Jul 5 10:41:34 UTC 2017
On Wed, 2017-07-05 at 11:24 +0200, Christian Ehrhardt wrote:
> On Wed, Jul 5, 2017 at 10:59 AM, Luca Boccassi <luca.boccassi at gmail.c
> > Just a note regarding the libs: if DPDK_LIBS is not used, how does
> > it get
> > the list of libraries to link to?
> It searches for -llibdpdk which will trigger the linker script
> the group of linking arguments.
> > Check if there's a LIBS variable, might be a better one to append
> > to
> > rather than LDFLAGS (it's also correct, the latter is for linker
> > flags, not
> > for libs).
> While that seems true to me as well I'd prefer to only change what we
> explain we need it for.
> In that case changing libs is not required so it would add complexity
> the change that is going to be discussed.
> But please feel free to suggest there.
> > Regarding fetching variables from pkgconfig it can be done in
> > autoconf,
> > I'll find how and forward it.
> Ok, in case we want/need to add those to libs that would be my way to
> Yet OTOH the linker script only adds the group and not the -lz and
> that the libs in pkg-config do.
> Not consuming projects for all the linking issues dpdk had in the
> sometimes mess with that - in the openvswitch case, they put all in a
> -whole statement.
> That ensures that all libs are linked, but since our pkg-config list
> more complete (longer) than the lindpdk linker script it might change
> things in an unwanted way.
> Another reason to not change the usage of libs unless needed I'd
> I just submitted to openvswitch development and set you on CC.
> Lets discuss there on the proposed change as needed - you might also
> able to help answering some questions that might come up.
I'm fine with just skipping it for the moment, and then maybe fix at a
later date, if it's not needed. Getting it to work out of the box is
For future reference, to query pkg-config in autoconf:
FOO=$($PKG_CONFIG --variable=libdir dpdk)
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 488 bytes
Desc: This is a digitally signed message part
More information about the Pkg-dpdk-devel