adding a runtime dependency into KDELIBS

Dario Freddi drf54321 at gmail.com
Thu Nov 24 00:18:49 GMT 2011


2011/11/24 Albert Astals Cid <aacid at kde.org>:
> El Dimecres, 23 de novembre de 2011, a les 00:00:29, Alex Fiestas va escriure:
>> We're in the process of merging a review which will partly fix the sad
>> situation of MTP/MPI/iPod devices in libsolid, the review I'm talking
>> about is:
>>
>> https://git.reviewboard.kde.org/r/103028/
>>
>> This as far as I know was (is) working with the linux-deprecated HAL
>> backend, so is something we urge to fix.
>>
>> This patch adds support for media-player-info which is basically what
>> replaces some features HAL has in the new u* stuff.
>>
>> So, Can I give the ship it for this patch? can we add an optional
>> runtime dependency to kdelibs 4.7.x ?
>
> I don't see this adding any kind of dependency from the POV of code, that is,
> it does not do qprocess nor the likes, so can you explain what the "optional
> runtime dependency" means here?

I think the code is accessing the DBus service, even though in the
patch this is not mentioned as well, so I'm slightly confused.
Besides, on a different topic, we should really standardize for
handling this kind of runtime dependencies, as I think they'll just
grow more as time goes

>
> Also Kevin's code looks like copied/duplicated "aka bad stuff".
>
> Albert
>
>>
>> Thanks.
>




More information about the kde-core-devel mailing list