Check the syntax of the rendered apparmor profile

canonical · Dec 15, 2023 · 7a07c40 · 7a07c40
1 parent 9fa0d98
commit 7a07c40
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/debian/control b/debian/control
@@ -7,6 +7,7 @@ Build-Depends: bash-completion,
                debianutils (>= 4.7),
                dh-python,
                dh-apparmor,
+               apparmor,
 # After debhelper 13.3 we no longer need dh-systemd.
 # On hirsute and later, dh-systemd doesn't even exist.
 # On recent releases, the first alternative will be used.

diff --git a/debian/rules b/debian/rules
@@ -64,6 +64,8 @@ override_dh_systemd_start:
 override_dh_auto_install:
 	dh_auto_install --destdir=debian/ubuntu-advantage-tools
 	debian/jinja2_render debian/apparmor/ubuntu_advantage_apt_news.jinja2 debian/apparmor/ubuntu_advantage_apt_news ubuntu_codename=${UBUNTU_CODENAME}
+	# quick syntax check on the generated profile
+	apparmor_parser -K -T -Q debian/apparmor/ubuntu_advantage_apt_news
 	install -D -m 644 $(CURDIR)/debian/apparmor/ubuntu_advantage_apt_news $(CURDIR)/debian/ubuntu-advantage-tools/etc/apparmor.d/ubuntu_advantage_apt_news
 	dh_apparmor -pubuntu-advantage-tools --profile-name=ubuntu_advantage_apt_news