FindLog4cplus.cmake broken about pion HOT 5 CLOSED

@Anticom can be Xenial has bumped version from 1.0.4 to 1.1.2, so it's possible.
Will check it later.
Also you can try change set(Log4Cplus_USE_STATIC_LIBS 1) to set(Log4Cplus_USE_STATIC_LIBS 0) in pion's CMakeLists.txt here https://github.com/splunk/pion/blob/develop/CMakeLists.txt#L119
It also can help you.

from pion.

@snikulov Indeed the version on my machine is 1.1.2. However I've looked at your FindLog4cplus.cmake and i don't see how this module is breaking anything on my machine. However Log4Cplus_USE_STATIC_LIBS being set to 1 by default looks like causing the issue for me.

Shouldn't find_path handle this on its own? Either way IMO it should be an option so the user has control over it.

from pion.

Shouldn't find_path handle this on its own? - Nop.
It specific for library authors. One adds S prefix, other add -static. No unified way.

Just remove Log4Cplus_USE_STATIC_LIBS at all from pion's CMakeLists.txt .
Then module will search dynamic by default, unless you explicitly not set it using -D

from pion.

Since we're building pion using Yocto i'd have to create a patch for this which would be unfortunate for such a trivial change. Is there any reason to not make it an option - besides eventually breaking working builds that use a static version of log4cplus - ?

If this is your only concern I'd vote for this "breaking" change.

from pion.

I think it is smaller then add option and use this option later.
Just remove one line and you'll give this option for user.

from pion.