Summary: | upstream apparmor profile | ||
---|---|---|---|
Product: | [Frameworks and Libraries] Akonadi | Reporter: | Jonathan Riddell <jr> |
Component: | server | Assignee: | kdepim bugs <kdepim-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | carl, sknauss |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: |
Description
Jonathan Riddell
2019-09-10 12:32:00 UTC
Well no, it doesn't make sense for the other distributions. Distro specific hacks shall be added by distributions (In reply to Christophe Giboudeaux from comment #1) > Well no, it doesn't make sense for the other distributions. > > Distro specific hacks shall be added by distributions You are right Distro specific hacks should not enter upstream. But this bug report is about getting AppArmor profile into upstream. I now found a solution to get rid of this symlink and we are able to express everything, we need in Apparmor directly, by using the Px to have a decidated mysql prfile for akonadi. Upps forgotten the link to Differential: https://phabricator.kde.org/D23908 "mysqld-akonadi" *is* a distro hack (In reply to Christophe Giboudeaux from comment #4) > "mysqld-akonadi" *is* a distro hack Have you looked at the Differential? There mysqld_akonadi is just a name for a mysqld profile, that is used by Akonadi, that has nothing to do with any Distribution. The initial comment from Jonathan had another distro hack with symlink. But having different profiles for one application is a common thing done in AppArmor and this is needed. As a mysqld runs globally needs other permissions than the instance created by Akonadi. Well, it's still only useful for debian and its various forks (ubuntu, neon...) (In reply to Christophe Giboudeaux from comment #6) > Well, it's still only useful for debian and its various forks (ubuntu, > neon...) No - it is useful for everyone who is using AppArmor or pinpoint me to parts, that are distro specific, so I can fix those parts. Closing as the relevant phabricator diff was merged |