On 02/03/2011 03:23 PM, Amos Jeffries wrote:
On 04/02/11 01:53, Tsantilas Christos wrote:
I will wait 1-2 hours if a new problem appear, and I will try to commit
the SMP SNMP patch again.
(thins patch was not lucky, we should drop it..... )


No pressure on that. It appears to have gotten well past the point of
initial failures now. The 1am and 2am bootstrap/sync have past. Another
2hrs should see full test completion on the snapshot system I ran early
manually.

I tried earlier today to see how many unit tests fail,...

* the ACL one just needed $(SNMP_LIBS) added to its LDADD list as expected.

In new patch the SNMP_LIBS included in COMMON_LIBS



* a followup one on testCacheManager is showing me what looks like a
circular dependency between libsnmp and libipc. libipc IIRC requires the
SNMP PDU send functions. SNMP requires the IPC forwarder stuff.

* testEvent, testEventLoop and testHttp* are still ahead and the usual
culprits for dependency hell.

It was a little tricky. Just adding the libsnmp on link libraries is not enough. If the libsnmp was linked before libipc then the Ipc::* objects missing. If the libipc linked before libsnmp then the Snmp::* objects missing.

To solve it I just add the src/tests/stub_ipc_Forwarder.cc file which is used to build testEvent*, testHttp* and testCacheManager:

---------------------------------------------
#include "ipc/Forwarder.h"

//Avoid linker errors about Ipc::Forwarder
void foo_stub_ipc_forwarder()
{
    Ipc::Forwarder foo(NULL,1.0);
}

---------------------------------------------

Regards,
    Christos



Amos

Reply via email to