interfaces/apparmor: remove unused apparmor variables

[zyga]

This patch removes unused apparmor variables APP_ID_DBUS and
APP_PKGNAME_DBUS which also removes the need to use .Developer in any
apparmor profile content.

Signed-off-by: Zygmunt Krynicki zygmunt.krynicki@canonical.com

[mvo5]

👍

[jdstrand]

Not sure why these are being removed now since we will have to add them immediately back for the bluez policy.

[zyga]

@jdstrand because bluez can use other variables. We cannot use .developer in any profile and we should not use .revision either. If you do that then you end up with one of the existing variables.

[jdstrand]

Note that .Developer isn't needed any more since this variables are the dbusified versions of the profile name (ie, what we substitute for ###PROFILEATTACH###). Please feel free to instead move these into modernVariables with an appropriate name.

[zyga]

@jdstrand ack, I'll land this branch, remove legacy variables and switch everything to modern variables. Sounds good?

[jdstrand]

That sounds fine.