adding a runtime dependency into KDELIBS

[Dario Freddi]

2011/11/24 Dario Freddi <drf54321 at gmail.com>:
> 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,

Forget this bit, I was just confusing myself, sorry.

>From the wiki page:

"media-player-info is a repository of data files describing media
player capabilities, mostly of mass-storage devices. These files
contain information about the directory layout to use to add music to
these devices, the supported file formats and so on.

These capabilities used to be provided by HAL in the
10-usb-music-players.fdi file, but HAL is now deprecated, so the
information is being provided as a separate package."

So it's likely the new properties will be provided reliably only if
that package is installed, which explains pretty much everything else.

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