Code review comment for lp:~alextu/lxc-android-config/lp1575564_include_rules_in_device_tarball

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

This is fine so long as this package provides /system/usr/share/apparmor/hardware/audio.d, /system/usr/share/apparmor/hardware/graphics.d and /system/usr/share/apparmor/hardware/video.d. If these directories are not guaranteed to exist and they don't for some reason, the apparmor policy will fail to load for all applications and they will fail to start (since the launcher won't be able to transition to the app's profile, because it isn't loaded in the kernel).

I would prefer that this merge add a debian/...dirs file to make this happen (this assumes that what this package ships isn't bind mounted over).

review: Needs Fixing

« Back to merge proposal