Differential D24664

[gmenu-dbusmenu-proxy] Don't create ~/.gtkrc-2.0
ClosedPublic
Actions

Authored by fvogt on Oct 15 2019, 1:51 PM.

Details

Reviewers
broulik
Commits
R120:ff84a4e5662e: [gmenu-dbusmenu-proxy] Don't create ~/.gtkrc-2.0
Summary

BUG: 412970

Test Plan

GTK-2.0 applications use breeze again. Global menu activation still works.

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
fvogt created this revision.Oct 15 2019, 1:51 PM
Restricted Application added a project: Plasma. · View Herald TranscriptOct 15 2019, 1:51 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
fvogt requested review of this revision.Oct 15 2019, 1:51 PM
fvogt edited the test plan for this revision. (Show Details)Oct 15 2019, 3:03 PM
broulik added a comment.Oct 15 2019, 5:01 PM

Since we watch the file, we will update it as neccessary once kconf_update has written it, right?
When it is deleted after that, we won't ever recreate it, though?

fvogt added a comment.Oct 15 2019, 5:03 PM
In D24664#547710, @broulik wrote:

Since we watch the file, we will update it as neccessary once kconf_update has written it, right?

Yup.

When it is deleted after that, we won't ever recreate it, though?

Yup.

broulik accepted this revision.Oct 16 2019, 6:54 AM

5.17 please

This revision is now accepted and ready to land.Oct 16 2019, 6:54 AM
Closed by commit R120:ff84a4e5662e: [gmenu-dbusmenu-proxy] Don't create ~/.gtkrc-2.0 (authored by fvogt). · Explain WhyOct 16 2019, 6:58 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mgmenu-dbusmenu-proxy/menuproxy.cpp (10 lines)

Diff 68016

View Options

gmenu-dbusmenu-proxy/menuproxy.cpp

Loading...