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
Branch
gtkfix
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 17712
Build 17730: arc lint + arc unit
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 67979

View Options

gmenu-dbusmenu-proxy/menuproxy.cpp

Loading...