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