07/12/2017 22:02, Ferruh Yigit: > On 11/30/2017 12:08 PM, Jasvinder Singh wrote: > > _LDLIBS-$(CONFIG_RTE_LIBRTE_FLOW_CLASSIFY) += -lrte_flow_classify > > _LDLIBS-$(CONFIG_RTE_LIBRTE_PIPELINE) += -lrte_pipeline > > +_LDLIBS-$(CONFIG_RTE_LIBRTE_TABLE) += --whole-archive > > _LDLIBS-$(CONFIG_RTE_LIBRTE_TABLE) += -lrte_table > > +_LDLIBS-$(CONFIG_RTE_LIBRTE_TABLE) += --no-whole-archive > > I think it is better to move rte_table between whole-archive flags, but I > vaguely remember Thomas preferred this in the past, so adding him for comment.
Yes it is good to keep the order, and insert whole-archive flags where necessary. Ideally we should not require these flags. Just checking: you need them for static link of libs while dynamically linking the PMDs, right?