D26900: Support multiple values in XDG_CURRENT_DESKTOP

David Faure noreply at phabricator.kde.org
Fri Jan 24 18:46:21 GMT 2020


dfaure created this revision.
dfaure added reviewers: apol, aacid.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
dfaure requested review of this revision.

REVISION SUMMARY
  The Desktop Entry spec says "If $XDG_CURRENT_DESKTOP is set then it contains a colon-separated list of strings."
  I found this bug (in a 2014 commit of mine) by simply reading this code and wondering why there was a QStringList.

TEST PLAN
  Unittest updated, fails without the fix

REPOSITORY
  R309 KService

BRANCH
  master

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

AFFECTED FILES
  autotests/kmimeassociationstest.cpp
  src/services/kservice.cpp

To: dfaure, apol, aacid
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200124/48201fcb/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list