Code review comment for lp:~phablet-team/media-hub/add-unity8-exception

Revision history for this message
Paweł Stołowski (stolowski) wrote :

Thanks for that Jim! There is one problem though, it throws early in Context constructor, because there is no package name set for unity8-dash:

apparmor profile name: unity8-dash, is_unconfined(): false, has_package_name(): false
terminate called after throwing an instance of 'std::logic_error'
  what(): apparmor::ubuntu::Context: Invalid profile name unity8-dash

If this is because unity8-dash doesn't come from a click package, then I think the ctor of Context also needs an exception for unity8-dash (making unity8-dash a .click is not in our plans).
Please let me know if there is any other way for me to make this constraint happy on my side.

« Back to merge proposal