[CRIU] [PATCH 23/24] compel cli: add libs command, use it

Kir Kolyshkin kir at virtuozzo.com
Mon Dec 19 22:32:52 PST 2016


On 12/19/2016 07:20 PM, Kir Kolyshkin wrote:
> On 12/19/2016 03:23 AM, Cyrill Gorcunov wrote:
>> On Mon, Dec 19, 2016 at 02:05:58PM +0300, Pavel Emelyanov wrote:
>>> On 12/19/2016 01:42 PM, Cyrill Gorcunov wrote:
>>>> On Mon, Dec 19, 2016 at 01:34:24PM +0300, Pavel Emelyanov wrote:
>>>>> On 12/17/2016 02:22 PM, Kir Kolyshkin wrote:
>>>>>> Add "compel libs" that prints the list of libraries needed
>>>>>> to link the parasite loader.
>>>>> O_o  What else other than libcompel.(a|so) will we have?
>>>> No other for now, but since all flags are plural it make
>>>> sense to define the same way here.
>>> My question was -- why do we need this action at all?!
>> I think this is convenient. Our compel-cli interface now is
>> very close to pkg-config utility.
>
> I might take a look into making pkg-config work for libcompel (at 
> least in installed case).
>
> Previously, I was stuck with these issues:
> (1) uninstalled case, which is used for criu and compel examples 
> (compel/tests/*);
> (2) two sets of libs needed -- one for parasite, one for parasite loader;
> (3) ability to specify add'l plugins for parasite.
>
> But it looks like (2) and (3) are solvable -- for all of it, we'll 
> just need multiple .pc files,
> like compel-parasite.pc, compel.pc, compel-plugin-$NAME.pc.
>
> I'll see if I can make this idea fly.

Having said that, I'd still ike the patches commited, as even in case 
we'll switch
to pkg-config (which is yet unclear), this patched contains a lot of 
preparatory
work and it's easier to base pkg-config work upon it.


More information about the CRIU mailing list