[Pacemaker] Linking lib/cib and lib/pengine to each other?

[Andrew Beekhof]

Give https://github.com/beekhof/pacemaker/commit/53c9122 a try

On Wed, Mar 27, 2013 at 7:43 AM, Viacheslav Dubrovskyi <dubrsl at gmail.com> wrote:
> 26.03.2013 19:41, Andrew Beekhof пишет:
>>>> Hi.
>>>>
>>>> I'm building a package for my distributive. Everything is built, but the
>>>> package does not pass our internal tests. I get errors like this:
>>>> verify-elf: ERROR: ./usr/lib/libpe_status.so.4.1.0: undefined symbol:
>>>> get_object_root
>> Was this the only undefined symbol?
> For  lib/cib -> lib/pengine yes.
> Another undefined symbols from lib/common and it's easy fixed.
>> It might be better to remove use of that function instead.
> Maybe yes, but I do not know what the developers were thinking when done so.
> For me as maintainer easy fix linkage.
>>
>>>> It mean, that libpe_status.so not linked with libcib.so where defined
>>>> get_object_root. I can easy fix it adding
>>>> libpe_status_la_LIBADD  =  $(top_builddir)/lib/cib/libcib.la
>>>> in lib/pengine/Makefile.am
>>>>
>>>> But for this I need build libcib before lib/pengine. And it's impossible
>>>> too, because libcib used symbols from lib/pengine. So we have situation,
>>>> when two library must be linked to each other.
>>>>
>>>> And this is very bad because then the in fact it should be one library.
>>>> Or symbols should be put in a third library, such as common.
>>>>
>>>> Can anyone comment on this situation?
>>> Patch for fix this error.
>>>
>>>
>
> --
> WBR,
> Viacheslav Dubrovskyi
>
>
> _______________________________________________
> Pacemaker mailing list: Pacemaker at oss.clusterlabs.org
> http://oss.clusterlabs.org/mailman/listinfo/pacemaker
>
> Project Home: http://www.clusterlabs.org
> Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
> Bugs: http://bugs.clusterlabs.org