[Differential] [Request, 21 lines] D3923: Make AppstreamQt optional

[davidedmundson (David Edmundson)]

davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  Normally I would just close any bug report saying a 3rd party lib should
  be optional, but it's quite a big dependency chain for a fairly minor
  feature that is easy to ifdef out.
  
  Also we're depending on something that was only released a few weeks ago, 
  in experience that generally will cause some distro to patch it out anyway.
  
  BUG: 374310

TEST PLAN
  My AppstreamQt (from distro packages) is broken, so needed to do a temporary workaround anyway.
  Built this and it works, haven't been able to test a with appstream version.

REPOSITORY
  R119 Plasma Desktop

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D3923

AFFECTED FILES
  CMakeLists.txt
  applets/kicker/CMakeLists.txt
  applets/kicker/plugin/appentry.cpp
  config-appstream.h.cmake

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, #plasma
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170102/3712cf52/attachment.html>