Hi Björn, This is indeed an error in the CMake file for discovery. At the moment it lists "/opt/local/include" and "/opt/local/lib" as include/link directory. Those can be removed. For the "includes" your proposed line is correct. The link libraries are already correct, the link_directories line is obsolete as it is.
Could you please open a bug for this and maybe attach a patch? We use Jira for issue tracking [1]. Ps: I see you are using the release, perhaps it is a good idea to use trunk. Small fixes like this can be made fairly simple and mostly are done when they pop up. Using trunk guarantees you don't miss it. [1]: http://issues.apache.org/jira/browse/CELIX 2013/1/11 Björn Petri <[email protected]> > Hello everyone, > > > As this is my first mail, hence I'll shortly introduce myself: I'm a > Bjoern Petri, a sofware engineer working for Thales and currently > investigating the capabilities of celix. > > Unfortunately I ran into some minor problems while building celix. More > precisely, it complains that it can't find the slp.h file although I set > the SLP_INCLUDE_DIR Option: > > [...] > [ 61%] Built target remote_service_admin > [ 61%] Building C object remote_services/discovery/** > CMakeFiles/discovery.dir/**private/src/discovery.c.o > /home/dm513/dev/machine_**service/celix-0.0.1-** > incubating/remote_services/**discovery/private/src/**discovery.c:29:17: > error: slp.h: No such file or directory > /home/dm513/dev/machine_**service/celix-0.0.1-** > incubating/remote_services/**discovery/private/src/**discovery.c:73: > error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before > ‘discovery_pollSLPCallback’ > /home/dm513/dev/machine_**service/celix-0.0.1-** > incubating/remote_services/**discovery/private/src/**discovery.c:74: > error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before > ‘discovery_attributesCallback’ > /home/dm513/dev/machine_**service/celix-0.0.1-** > incubating/remote_services/**discovery/private/src/**discovery.c:77: > error: expected ‘)’ before ‘hslp’ > /home/dm513/dev/machine_**service/celix-0.0.1-** > incubating/remote_services/**discovery/private/src/**discovery.c: In > function ‘discovery_deregisterEndpoint’**: > [...] > > I could fix that by adding "include_directories(${SLP_**INCLUDE_DIRS})" > to remote_services/discovery/**CMakeLists.txt. > But I am not sure whether this is the correct solution. Maybe you could > give me a little hint?! > > Regards, > Bjoern > -- Met vriendelijke groet, Alexander Broekhuis
