Update `libdbusmenuqt` and the AppMenu plugin
Needs ReviewPublic
Actions

Authored by ntninja on Feb 3 2019, 2:42 PM.

Details

Reviewers

Zren
majohnson

Summary

What the title says, synchronize the relevant bits of source code with upstream (plasma-workspace).

Lint

Lint Skipped

Unit

Unit Tests Skipped

Restricted Application added a subscriber: plasma-devel. · View Herald Transcript

ntninja requested review of this revision.Feb 3 2019, 2:42 PM

plugin/libdbusmenuqt/CMakeLists.txt
27 ↗	(On Diff #50770)	We do not bundle the libdbusmenuqt test directory in this repo. So you need to comment this line out otherwise it will not build with the test directory missing.

If this fixes bug 401725, please add a line "BUG: 401725" to the commit message.

@Zren: Updated! Thanks for pointing this out!

@cfeck: It contains the fix from bug 399975 yes, but I don't know whether it will fix LyX.

			Path	Packages
M			plugin/appmenumodel.h (38 lines)
M			plugin/appmenumodel.cpp (138 lines)
M			plugin/appmenuplugin.cpp (3 lines)
M			plugin/libdbusmenuqt/dbusmenuimporter.cpp (4 lines)