Differential D8239

Use correct .desktop file name when setting KMail as default
ClosedPublic
Actions

Authored by dvratil on Oct 10 2017, 4:38 PM.

Details

Reviewers
davidedmundson
Group Reviewers
Plasma
Commits
R119:ca72cd44ebc0: Use correct .desktop file name when setting KMail as default
Summary

Since Applications 17.08 KMail's desktop file is called org.kde.kmail2.desktop
in order to match its DBus name. This breaks setting KMail as a default
application, since org.kde.kmail.desktop no longer exists. This change checks
for existence of org.kde.kmail2.desktop file and falls back to using the
old name.

Diff Detail

Repository
R119 Plasma Desktop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
dvratil created this revision.Oct 10 2017, 4:38 PM
Restricted Application added a project: Plasma. · View Herald TranscriptOct 10 2017, 4:38 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson accepted this revision.Oct 10 2017, 4:40 PM
davidedmundson added a subscriber: davidedmundson.

is a kconf_update needed?

This revision is now accepted and ready to land.Oct 10 2017, 4:40 PM
Closed by commit R119:ca72cd44ebc0: Use correct .desktop file name when setting KMail as default (authored by dvratil). · Explain WhyOct 10 2017, 4:52 PM
This revision was automatically updated to reflect the committed changes.
dvratil added a comment.Oct 10 2017, 4:53 PM
In D8239#154122, @davidedmundson wrote:

is a kconf_update needed?

Added kconf_update script to KMail repo

Revision Contents
Changeset List

Diff 20578

View Options

kcms/componentchooser/componentchooseremail.cpp

Loading...