Bug#923213: squid: Please disable the "Test apparmor" autopkgtest

2019-03-08 Thread Paul Gevers
Hi, On 08-03-2019 09:38, Amos Jeffries wrote: > On 8/03/19 6:58 am, Paul Gevers wrote: >> On 07-03-2019 17:51, Luigi Gangitano wrote: >>> 4.6-2 has just been uploaded with a fix for 923213. I really appreciate >>> the opportunity to let it in Buster. >> >> Please file an unblock bug. I have one

Bug#923213: squid: Please disable the "Test apparmor" autopkgtest

2019-03-08 Thread Amos Jeffries
On 8/03/19 6:58 am, Paul Gevers wrote: > Luigi, > > On 07-03-2019 17:51, Luigi Gangitano wrote: >> 4.6-2 has just been uploaded with a fix for 923213. I really appreciate >> the opportunity to let it in Buster. > > Please file an unblock bug. I have one question about the proposed > solution:

Bug#923213: squid: Please disable the "Test apparmor" autopkgtest

2019-03-07 Thread Paul Gevers
Luigi, On 07-03-2019 17:51, Luigi Gangitano wrote: > 4.6-2 has just been uploaded with a fix for 923213. I really appreciate > the opportunity to let it in Buster. Please file an unblock bug. I have one question about the proposed solution: you preinst doesn't honor local changes if the package

Bug#923213: squid: Please disable the "Test apparmor" autopkgtest

2019-03-07 Thread Luigi Gangitano
Hi Paul, > On Mon, 25 Feb 2019 07:31:42 +0100 intrig...@debian.org > wrote: >> If I got it right, on Debian we don't ship >> /etc/apparmor.d/usr.sbin.squid, so none of the tests in the >> test_zz_apparmor function are meaningful in this context. >> So I think this

Bug#923213: squid: Please disable the "Test apparmor" autopkgtest

2019-03-07 Thread Paul Gevers
Hi all, On Mon, 25 Feb 2019 07:31:42 +0100 intrig...@debian.org wrote: > If I got it right, on Debian we don't ship > /etc/apparmor.d/usr.sbin.squid, so none of the tests in the > test_zz_apparmor function are meaningful in this context. > So I think this entire test case should be skipped on

Bug#923213: squid: Please disable the "Test apparmor" autopkgtest

2019-02-24 Thread intrigeri
Source: squid Severity: normal Version: 4.6-1 Hi, it seems that test_zz_apparmor only passed on Debian previously because it implicitly relied on a bug in apparmor_parser, which used to return 0 if called on a non-existent profile. This bug was fixed in apparmor 2.13.2-8, which broke squid's