Support oxygen gtk theme in abstractions/gnome

Bug #914382 reported by Felix Geyer
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
apparmor (Ubuntu)
Incomplete
Wishlist
Felix Geyer

Bug Description

Kubuntu uses the oxygen gtk theme by default.
It reads some kde config files and icons and calls kde4-config.
This is not yet covered by abstractions/gnome.

Attaching a patch that adds the necessary rules.
The /usr/share/kubuntu-*-settings/{kde4-profile/default/,} globbing is required because there are a few kubuntu settings packages that add their path to the kde config search paths.
For example
/usr/share/kubuntu-default-settings/kde4-profile/default/share/config/kdeglobals
/usr/share/kubuntu-netbook-default-settings/share/config/kdeglobals
/usr/share/kubuntu-low-fat-settings/share/config/kdeglobals

I have also created a profile for kde4-config but I'm not sure if that's suitable to be included in apparmor.

Revision history for this message
Felix Geyer (debfx) wrote :
Revision history for this message
Felix Geyer (debfx) wrote :
tags: added: patch
Revision history for this message
Jamie Strandboge (jdstrand) wrote :

Thanks for the patch! I'm uncomfortable with adding this to the gnome abstraction. IMHO this should be in the kde abstraction. Also, we will be moving away from using Ux in shipped abstractions (patches pending), so I am NAKing the 'kde4-config Ux' part. Personally, I would be open to a child profile in the ked abstration for it, but I think ultimately this needs to be discussed on the mailing list.

Changed in apparmor (Ubuntu):
assignee: nobody → Felix Geyer (debfx)
status: New → In Progress
Revision history for this message
Felix Geyer (debfx) wrote :

> I'm uncomfortable with adding this to the gnome abstraction. IMHO this should be in the kde abstraction.

I tihink it doesn't make sense to add it to the kde abstraction since KDE applications don't use the gtk theme.
To work out of the box for kde/kubuntu users the rules need to be in gnome.
If that's not acceptable maybe it should go into an own abstraction.

Revision history for this message
Jamie Strandboge (jdstrand) wrote :

Is this patch still valid on 14.04 or 14.10?

tags: added: aa-policy
Changed in apparmor (Ubuntu):
status: In Progress → Incomplete
Changed in apparmor (Ubuntu):
importance: Undecided → Wishlist
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.