<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 5, 2017 at 10:59 AM, Luca Boccassi <span dir="ltr"><<a href="mailto:luca.boccassi@gmail.com" target="_blank">luca.boccassi@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":8b8" class="a3s aXjCH m15d11fad5b2e3a7a">Just a note regarding the libs: if DPDK_LIBS is not used, how does it get the list of libraries to link to?<br></div></blockquote><div><br></div><div>It searches for -llibdpdk which will trigger the linker script generating the group of linking arguments. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":8b8" class="a3s aXjCH m15d11fad5b2e3a7a">
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).<br></div></blockquote><div><br></div><div>While that seems true to me as well I'd prefer to only change what we can explain we need it for.</div><div>In that case changing libs is not required so it would add complexity to the change that is going to be discussed.</div><div>But please feel free to suggest there.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":8b8" class="a3s aXjCH m15d11fad5b2e3a7a">
Regarding fetching variables from pkgconfig it can be done in autoconf, I'll find how and forward it.</div></blockquote></div><div class="gmail_extra"><br></div>Ok, in case we want/need to add those to libs that would be my way to go.</div><div class="gmail_extra">Yet OTOH the linker script only adds the group and not the -lz and others that the libs in pkg-config do.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Not consuming projects for all the linking issues dpdk had in the past sometimes mess with that - in the openvswitch case, they put all in a -whole statement.</div><div class="gmail_extra">That ensures that all libs are linked, but since our pkg-config list is more complete (longer) than the lindpdk linker script it might change things in an unwanted way.</div><div class="gmail_extra">Another reason to not change the usage of libs unless needed I'd think.<br><br>I just submitted to openvswitch development and set you on CC.</div><div class="gmail_extra">Lets discuss there on the proposed change as needed - you might also be able to help answering some questions that might come up.</div><div class="gmail_extra"><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><span style="color:rgb(136,136,136);font-size:12.8px">Christian Ehrhardt</span><div style="color:rgb(136,136,136);font-size:12.8px">Software Engineer, Ubuntu Server</div><div style="color:rgb(136,136,136);font-size:12.8px">Canonical Ltd</div></div></div></div></div>
</div></div>