interfaces: allow unity7 launcher API #1173

Merged
merged 10 commits into from May 19, 2016

Conversation

Projects
None yet
4 participants
Contributor

jdstrand commented May 16, 2016

No description provided.

jdstrand added some commits May 12, 2016

interfaces: make gmenu access non-application specific (eg, fixes lib…
…reoffice)

The gmenu DBus api was not designed for application isolation and apps may
specify anything as their 'path'. For example, these work in the many cases:
- /org/gtk/Application/anonymous{,/**}
- /com/canonical/unity/gtk/window/[0-9]*

but libreoffice does:
- /org/libreoffice{,/**}

As such, we cannot mediate by DBus path so be as strict as we can in the other
mediated parts.
Contributor

zyga commented May 17, 2016

+1

I wonder if there's a way to generate some of those rules and definitions. It certainly would be nicer if we could just cross-check this with some API docs.

Collaborator

mvo5 commented May 19, 2016

👍

Collaborator

mvo5 commented May 19, 2016

retest this please

@niemeyer niemeyer merged commit 9893f60 into snapcore:master May 19, 2016

4 checks passed

Integration tests Success
Details
autopkgtest Success
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 79.526%
Details

@jdstrand jdstrand deleted the jdstrand:unity-launcher-updates branch May 23, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment