Bug#888038: chrony: please add AppArmor profile for chronyd

2018-01-23 Thread Jamie Strandboge
On Tue, 2018-01-23 at 17:37 +0100, Vincent Blut wrote: ... > I just noticed that you did not include the debhelper token in the > preinst script. It is necessary to allow debhelper to substitute it > with > the information present in the “chrony.maintscript” file. > > Git formatted patch

Bug#888038: chrony: please add AppArmor profile for chronyd

2018-01-23 Thread Vincent Blut
Hey Jamie, On Mon, Jan 22, 2018 at 02:17:26PM -0600, Jamie Strandboge wrote: diff -Nru chrony-3.2/debian/preinst chrony-3.2/debian/preinst --- chrony-3.2/debian/preinst 1969-12-31 18:00:00.0 -0600 +++ chrony-3.2/debian/preinst 2018-01-22 13:48:21.0 -0600 @@ -0,0 +1,31 @@

Bug#888038: chrony: please add AppArmor profile for chronyd

2018-01-22 Thread Vincent Blut
On Mon, Jan 22, 2018 at 04:47:46PM -0600, Jamie Strandboge wrote: I think we should prefix /run/* paths by /{,var/} to make our profile easier to port to other distros as some of them have yet to migrate from /var/run to /run. This sounds fine too. Note that your profile has this:

Bug#888038: chrony: please add AppArmor profile for chronyd

2018-01-22 Thread Jamie Strandboge
On Mon, 2018-01-22 at 22:55 +0100, Vincent Blut wrote: > Hi Jamie, > > On Mon, Jan 22, 2018 at 02:17:26PM -0600, Jamie Strandboge wrote: > > Package: chrony > > Version: 3.2-1 > > Severity: wishlist > > Tags: patch > > User: ubuntu-de...@lists.ubuntu.com > > Usertags: origin-ubuntu bionic

Bug#888038: chrony: please add AppArmor profile for chronyd

2018-01-22 Thread Vincent Blut
Hi Jamie, On Mon, Jan 22, 2018 at 02:17:26PM -0600, Jamie Strandboge wrote: Package: chrony Version: 3.2-1 Severity: wishlist Tags: patch User: ubuntu-de...@lists.ubuntu.com Usertags: origin-ubuntu bionic ubuntu-patch Dear Maintainer, In Ubuntu, the attached patch was applied to achieve the

Bug#888038: chrony: please add AppArmor profile for chronyd

2018-01-22 Thread Jamie Strandboge
Package: chrony Version: 3.2-1 Severity: wishlist Tags: patch User: ubuntu-de...@lists.ubuntu.com Usertags: origin-ubuntu bionic ubuntu-patch Dear Maintainer, In Ubuntu, the attached patch was applied to achieve the following: * add AppArmor profile for /usr/sbin/chronyd: - add