Shared library build broken

I just noticed that shared library build is broking. I am building current master. I had to make this change to get it to build:

(quoted diff) +1 -1

-CONFIG_RTE_LIBRTE_PMD_BOND=y
+CONFIG_RTE_LIBRTE_PMD_BOND=n

One of the recent bonding commits broke some dependencies I think but I didn't investigate further.

test_link_bonding.o: In function `test_add_slave_to_bonded_device':
test_link_bonding.c:(.text+0x44a): undefined reference to rte_eth_bond_slave_add' test_link_bonding.c:(.text+0x462): undefined reference to rte_eth_bond_slaves_get' test_link_bonding.c:(.text+0x487): undefined reference to `rte_eth_bond_active_slaves_get
....

--TFH

Sorry,

I apologize on behalf of my fingers. I meant combined library build is broken when PMD_BOND is selected.

Quoted 19 lines · Thomas F Herbert

I just noticed that shared library build is broking. I am building current master. I had to make this change to get it to build:

(quoted diff) +1 -1

-CONFIG_RTE_LIBRTE_PMD_BOND=y
+CONFIG_RTE_LIBRTE_PMD_BOND=n

One of the recent bonding commits broke some dependencies I think but I didn't investigate further.

test_link_bonding.o: In function `test_add_slave_to_bonded_device':
test_link_bonding.c:(.text+0x44a): undefined reference to rte_eth_bond_slave_add' test_link_bonding.c:(.text+0x462): undefined reference to rte_eth_bond_slaves_get' test_link_bonding.c:(.text+0x487): undefined reference to `rte_eth_bond_active_slaves_get
....

Quoted 1 line

--TFH

Quoted 26 lines · Thomas F Herbert

Sorry,

I apologize on behalf of my fingers. I meant combined library build is broken when PMD_BOND is selected.

Quoted 19 lines · Thomas F Herbert

I just noticed that shared library build is broking. I am building current master. I had to make this change to get it to build:

(quoted diff) +1 -1

-CONFIG_RTE_LIBRTE_PMD_BOND=y
+CONFIG_RTE_LIBRTE_PMD_BOND=n

One of the recent bonding commits broke some dependencies I think but I didn't investigate further.

test_link_bonding.o: In function `test_add_slave_to_bonded_device':
test_link_bonding.c:(.text+0x44a): undefined reference to rte_eth_bond_slave_add' test_link_bonding.c:(.text+0x462): undefined reference to rte_eth_bond_slaves_get' test_link_bonding.c:(.text+0x487): undefined reference to `rte_eth_bond_active_slaves_get
....

Quoted 1 line

Quoted 1 line

--TFH

I just sent a patch to fix the issue.

Drivers (PMDs) were not being archive in the combined library.

Sergio