* qtelepathy-qt5:

debian/rules:
- any change someone spend some time to look at the races in the failing tests 
and run them? (at least as adt tests).
- optional, but please consider using dh_install --fail-missing instead of 
--list-missing and rm in debian/tmp the file you don't want to install, that 
helps controlling and ensuring we are building everything we need.

debian/control: it's weird to have a build-depend as a metapackage
(qt5-default), didn't we evolve the policy for other Qt5 packages IIRC,
like depending on qtbase5-dev?

Otherwise, this package looks good.

* media-hub:
debian/control:
# We rely on C++11 features, and to prevent from ABI breaks
# in libstdc++ causing us issues, we explicitly select a G++
# version.
               g++-4.9,
-> is that still the case nowdays? I thought g++ stabilized their C++11 
features ABI-break wise. Note for myself: this is also hardcoded in 
debian/rules.

Package: media-hub pre-depends on dpkg, but isn't multi-arch or any
postinst use any "more recent" dpkg features, can that be cleaned up? (I
think even the other pre-depends on libmedia-hub* can be cleaned up now
that trusty is out)

I assume the security team already got a security pass at it?

Apart from those nitpicks, looking good, keep me posted about the
questions :)

** Changed in: media-hub (Ubuntu)
       Status: New => Incomplete

** Changed in: telepathy-qt5 (Ubuntu)
       Status: New => Incomplete

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1427958

Title:
  [MIR] media-hub

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/media-hub/+bug/1427958/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to