address-book-app

Merge lp:~ken-vandine/address-book-app/add_profile into lp:address-book-app

  1. add_profile
  2. Merge into trunk
Proposed by Ken VanDine
Status: Merged
Approved by: Renato Araujo Oliveira Filho
Approved revision: 651
Merged at revision: 670
Proposed branch: lp:~ken-vandine/address-book-app/add_profile
Merge into: lp:address-book-app
Diff against target: 57 lines (+22/-0)
4 files modified
debian/address-book-app.install (+1/-0)
debian/apparmor/address-book-app (+15/-0)
debian/control.in (+1/-0)
debian/rules (+5/-0)
To merge this branch: bzr merge lp:~ken-vandine/address-book-app/add_profile
Reviewer Review Type Date Requested Status
Renato Araujo Oliveira Filho (community) Approve
system-apps-ci-bot continuous-integration Approve
Review via email: mp+312279@code.launchpad.net

Commit message

added apparmor profile for address-book-app

Description of the change

added apparmor profile for address-book-app

To post a comment you must log in.
Revision history for this message
system-apps-ci-bot (system-apps-ci-bot) wrote :

FAILED: Continuous integration, rev:650
https://jenkins.canonical.com/system-apps/job/lp-address-book-app-ci/47/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/system-apps/job/build/2100/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-0-fetch/2103
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1930
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1930/artifact/output/*zip*/output.zip
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1930/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=zesty/1930/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1930
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1930/artifact/output/*zip*/output.zip
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1930/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=zesty/1930/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1930
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1930/artifact/output/*zip*/output.zip
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1930/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=zesty/1930/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-apps/job/lp-address-book-app-ci/47/rebuild

review: Needs Fixing (continuous-integration)
lp:~ken-vandine/address-book-app/add_profile updated
651. By Ken VanDine

build dep on dh-apparmor

Revision history for this message
system-apps-ci-bot (system-apps-ci-bot) wrote :

PASSED: Continuous integration, rev:651
https://jenkins.canonical.com/system-apps/job/lp-address-book-app-ci/48/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build/2101
    SUCCESS: https://jenkins.canonical.com/system-apps/job/test-0-autopkgtest/label=phone-armhf,release=vivid+overlay,testname=default/517
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-0-fetch/2104
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1932
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1932/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1932
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1932/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=zesty/1932
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=zesty/1932/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1932
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1932/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1932
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1932/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=zesty/1932
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=zesty/1932/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1932
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1932/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1932
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1932/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=zesty/1932
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=zesty/1932/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-apps/job/lp-address-book-app-ci/48/rebuild

review: Approve (continuous-integration)
Revision history for this message
Renato Araujo Oliveira Filho (renatofilho) wrote :

looks good to me.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/address-book-app.install'
--- debian/address-book-app.install 2014-06-25 22:02:31 +0000
+++ debian/address-book-app.install 2016-12-01 18:14:00 +0000
@@ -5,3 +5,4 @@
5usr/share/url-dispatcher/urls/5usr/share/url-dispatcher/urls/
6usr/share/content-hub/peers/address-book-app6usr/share/content-hub/peers/address-book-app
7usr/share/locale/*/LC_MESSAGES/address-book-app.mo7usr/share/locale/*/LC_MESSAGES/address-book-app.mo
8debian/apparmor/address-book-app etc/apparmor.d
89
=== added directory 'debian/apparmor'
=== added file 'debian/apparmor/address-book-app'
--- debian/apparmor/address-book-app 1970-01-01 00:00:00 +0000
+++ debian/apparmor/address-book-app 2016-12-01 18:14:00 +0000
@@ -0,0 +1,15 @@
1# vim:syntax=apparmor
2#include <tunables/global>
3
4# Mostly unconfined. Don't allow capability or any mount rules. Also ensure that
5# exec inherits from this profile
6profile address-book-app (attach_disconnected) {
7 network,
8 / rwkl,
9 /** rwlkm,
10 /** pix,
11 dbus,
12 signal,
13 ptrace,
14 unix,
15}
016
=== modified file 'debian/control.in'
--- debian/control.in 2016-11-17 11:54:15 +0000
+++ debian/control.in 2016-12-01 18:14:00 +0000
@@ -5,6 +5,7 @@
5Build-Depends: cmake,5Build-Depends: cmake,
6 dbus,6 dbus,
7 debhelper (>= 9),7 debhelper (>= 9),
8 dh-apparmor,
8 dh-translations,9 dh-translations,
9 libgl1-mesa-dev | libgl-dev,10 libgl1-mesa-dev | libgl-dev,
10 libgles2-mesa-dev,11 libgles2-mesa-dev,
1112
=== modified file 'debian/rules'
--- debian/rules 2016-07-22 15:22:45 +0000
+++ debian/rules 2016-12-01 18:14:00 +0000
@@ -8,6 +8,11 @@
8%:8%:
9 dh $@ --parallel --fail-missing --with translations9 dh $@ --parallel --fail-missing --with translations
1010
11override_dh_auto_install:
12 dh_auto_install
13 dh_apparmor -paddress-book-app --profile-name=address-book-app
14
15
11override_dh_translations:16override_dh_translations:
12 # Override dh_translations to work around http://pad.lv/1183262.17 # Override dh_translations to work around http://pad.lv/1183262.
13 # Unfortunately, the default build directory is not exposed18 # Unfortunately, the default build directory is not exposed

Subscribers

People subscribed via source and target branches