On 03/02/2016 02:40 PM, Thomas Monjalon wrote:

2016-03-02 14:30, Panu Matilainen:

quoted

On 03/01/2016 04:48 PM, Panu Matilainen wrote:

quoted

On 03/01/2016 03:40 PM, Thomas Monjalon wrote:

quoted

ping
I would like to be sure nothing is forgotten in this new revision.

Sorry, didn't realize you were waiting for input from me, it feels a bit
strange to comment on something supposedly coming from myself :)

quoted

2016-02-23 23:20, Thomas Monjalon:

quoted

From: Panu Matilainen <redacted>

The physically linked-together combined library has been an increasing
source of problems, as was predicted when library and symbol versioning
was introduced. Replace the complex and fragile construction with a
simple linker script which achieves the same without all the problems,
remove the related kludges from eg mlx drivers.

Since creating the linker script is practically zero cost, remove the
config option and just create it always.

Based on a patch by Sergio Gonzales Monroy, linker script approach
initially suggested by Neil Horman.

Suggested-by: Sergio Gonzalez Monroy <redacted>
Suggested-by: Neil Horman <nhorman@tuxdriver.com>
Signed-off-by: Panu Matilainen <redacted>
Signed-off-by: Thomas Monjalon <redacted>
---
v2:
- move RTE_LIBNAME assignment rte.vars.mk to rte.combinedlib.mk
- update crypto
- update doc
- update rte.app.mk
- update test-build.sh

Briefly tested, gets generated and installed as it should etc - looks
good to me.

Forgot to note that the patch doesn't apply anymore because of
scripts/test-build.sh changes, so it needs a rebase. Want me to send a
v3 or will you handle it when committing?

On a related note, if this is about to go in then I'd rather have it
sooner than later because it also conflicts with the LDLIBS fixing
that's been slowly going on for months and months but been on hold
lately, partly because of this hangup.

Applied, thanks

Awesome, thank you! :)

	- Panu -