D9850: Remove redundant changeLog field

Nathaniel Graham noreply at phabricator.kde.org
Sat Jan 13 04:25:26 UTC 2018


ngraham created this revision.
ngraham added reviewers: apol, Discover Software Store.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
ngraham requested review of this revision.

REVISION SUMMARY
  BUG: 387041
  
  For Debian packages, PackageKit manufactures the `updateText` and `updates` fields from the package's changelog. PackageKit gives us the `updateText` and `updates` fields, but also includes the original text used to synthesize them in a field called `changeLog`.
  
  So we should only display `updateText` and `updates`, since `changeLog` is just the the raw information used to synthesize them, and if we show it too, there's duplicate information.
  
  For more information about this, see https://github.com/hughsie/PackageKit/issues/227

TEST PLAN
  This change needs testing from people who are not using Debian-based distros to make sure it didn't break anything for them!
  
  I tested in KDE Neon, a Debian-based distro. Before:
  
  After:

REPOSITORY
  R134 Discover Software Store

BRANCH
  master

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

AFFECTED FILES
  libdiscover/backends/PackageKitBackend/PackageKitResource.cpp

To: ngraham, apol, #discover_software_store
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180113/5dfefe24/attachment.html>


More information about the Plasma-devel mailing list