hard coded dependency on non-existing version of runtime library

[Matthias Klose]

you really should get these packages into MAIN, following a proper MIR process. It's insane that such kind of issues are still get unnoted for uploads.

hard coded dependency on non-existing version of runtime library

Package: qtubuntu-media
Section: libs
Architecture: i386 amd64 armhf
Conflicts: mediaplayerplugin-aal
Replaces: mediaplayerplugin-aal
Provides: mediaplayerplugin-aal
Depends: ${misc:Depends},
         ${shlibs:Depends},
         qtvideonode-plugin,
         libmediascanner-2.0-3
Description: QtMultimedia plugin for AAL+
 Video and audio playback plugin that interfaces Qt with
 the hybris media backend on Android.

[Jim Hodapp (jhodapp)]

Can you point me to any docs on the process of requesting this?

[Matthias Klose (doko)]

https://wiki.ubuntu.com/MainInclusionProcess
https://wiki.ubuntu.com/UbuntuMainInclusionRequirements

[James Henstridge (jamesh)]

This is a bit of a weird one, since none of the libraries in the qtubuntu-media package actually link to libmediascanner:

    phablet@ubuntu-phablet:~$ dpkg -L qtubuntu-media | grep .so
    /usr/lib/arm-linux-gnueabihf/qt5/plugins/mediaservice/libaalmediaplayer.so
    /usr/lib/arm-linux-gnueabihf/qt5/imports/Ubuntu/Media/libubuntumediaplugin.so
    phablet@ubuntu-phablet:~$ ldd /usr/lib/arm-linux-gnueabihf/qt5/plugins/mediaservice/libaalmediaplayer.so | grep mediascanner
    phablet@ubuntu-phablet:~$ ldd /usr/lib/arm-linux-gnueabihf/qt5/imports/Ubuntu/Media/libubuntumediaplugin.so | grep mediascanner
    phablet@ubuntu-phablet:~$

While there is certainly some code in the tree using the mediascanner APIs:

http://bazaar.launchpad.net/~phablet-team/qtubuntu-media/trunk/view/head:/src/aal/aalmetadatareadercontrol.cpp#L81

That file doesn't appear to be listed in the sources of qmake project file for the plugin:

http://bazaar.launchpad.net/~phablet-team/qtubuntu-media/trunk/view/head:/src/aal/aal.pro

While the plugin is being linked with -lmediascanner-2.0, that dependency is probably ignored due to everything in Ubuntu being linked using the --as-needed linker flag.

Is it possible that someone added the explicit dependency on libmediascanner-2.0-3 because they didn't understand why it wasn't being added automatically?

[Matthias Klose (doko)]

can't comment on the intended linkage, but apparently it is not needed. If you still want to add a dependency, see the packaging of the libxml2 package, how the suggestion of the icu package is created.

Package: libxml2-dbg
Depends: libxml2 (= ${binary:Version}), ${dep:libicudbg}, ${misc:Depends}

override_dh_gencontrol:
        dh_gencontrol -- -Vdep:libicudbg="`dpkg-query -f '$${Depends}' -W libicu-dev | sed 's/.*\(libicu[0-9]*\).*/\1/'`-dbg"

[Jim Hodapp (jhodapp)]

This fix is part of silo 55 which QA decided will land right after the OTA7 snapshot is taken.

[James Henstridge (jamesh)]

Hi Jim,

What's the status of this now? I assume this is the landing you were referring to:

https://requests.ci-train.ubuntu.com/#/ticket/269

I see that it is landed late last week, but it looks like it was vivid only. Also, I can still see the spurious libmediascanner-2.0-3 dependency for the new qtubuntu-media packages in the overlay PPA (not that it matters for vivid, but it does indicate that the changes haven't landed).

[Jim Hodapp (jhodapp)]

This is in the wily overlay PPA now, full fix.

[Iain Lane (laney)]

On Tue, Oct 20, 2015 at 01:03:54PM -0000, Jim Hodapp wrote:
> This is in the wily overlay PPA now, full fix.

Thanks, sorry I forgot to ping you when doing a moar minimal one
yesterday (was trying to get the new mediascanner to go into wily before
we fully freeze it)

--
Iain Lane [ <email address hidden> ]
Debian Developer [ <email address hidden> ]
Ubuntu Developer [ <email address hidden> ]

[Launchpad Janitor (janitor)]

This bug was fixed in the package qtubuntu-media - 0.7.1+15.10.20151020.1-0ubuntu1

---------------
qtubuntu-media (0.7.1+15.10.20151020.1-0ubuntu1) wily; urgency=medium

  [ Alfonso Sanchez-Beato ]
  * Make sure the that the current player instance is controlable by
    MPRIS controls if the type of player is appropriate for playlist
    control. Also, don't add a track to the tracklist when
    AalMediaPlayerService::setMedia() is called. (LP: #1500859)

  [ Jim Hodapp ]
  * Make sure the that the current player instance is controlable by
    MPRIS controls if the type of player is appropriate for playlist
    control. Also, don't add a track to the tracklist when
    AalMediaPlayerService::setMedia() is called. (LP: #1500859)

  [ CI Train Bot ]
  * No-change rebuild.

 -- Alfonso Sanchez-Beato <email address hidden> Tue, 20 Oct 2015 13:10:40 +0000