Re: [apparmor] [parser patch] fix apparmor cache tempfile location to use passed arg v2

2013-07-26 Thread Steve Beattie
On Thu, Jul 25, 2013 at 07:17:19PM -0700, Seth Arnold wrote: Acked-by: Seth Arnold seth.arn...@canonical.com In case people were wondering why I hadn't committed this yet, I self-NACKed my patch because I had committed the unpardonable sin of not writing a testcase to demonstrate the issue and

Re: [apparmor] [parser patch] fix apparmor cache tempfile location to use passed arg v2

2013-07-26 Thread John Johansen
On 07/25/2013 11:57 PM, Steve Beattie wrote: On Thu, Jul 25, 2013 at 07:17:19PM -0700, Seth Arnold wrote: Acked-by: Seth Arnold seth.arn...@canonical.com In case people were wondering why I hadn't committed this yet, I self-NACKed my patch because I had committed the unpardonable sin of not

Re: [apparmor] [parser patch] fix apparmor cache tempfile location to use passed arg v2

2013-07-26 Thread Seth Arnold
On Thu, Jul 25, 2013 at 11:57:56PM -0700, Steve Beattie wrote: On Thu, Jul 25, 2013 at 07:17:19PM -0700, Seth Arnold wrote: Acked-by: Seth Arnold seth.arn...@canonical.com In case people were wondering why I hadn't committed this yet, I self-NACKed my patch because I had committed the

Re: [apparmor] [parser patch] fix apparmor cache tempfile location to use passed arg

2013-07-25 Thread Seth Arnold
On Tue, Jul 23, 2013 at 08:36:12AM -0700, Steve Beattie wrote: Subject: [parser patch] fix apparmor cache tempfile location to use passed arg This patch fixes problems in the handling of both the final cache name location and the temporary cache file when an alternate location is specified

Re: [apparmor] [parser patch] fix apparmor cache tempfile location to use passed arg

2013-07-24 Thread John Johansen
On 07/23/2013 10:43 AM, Tyler Hicks wrote: One minor nitpick/question... On 2013-07-23 08:36:12, Steve Beattie wrote: Subject: [parser patch] fix apparmor cache tempfile location to use passed arg This patch fixes problems in the handling of both the final cache name location

[apparmor] [parser patch] fix apparmor cache tempfile location to use passed arg

2013-07-23 Thread Steve Beattie
Subject: [parser patch] fix apparmor cache tempfile location to use passed arg This patch fixes problems in the handling of both the final cache name location and the temporary cache file when an alternate location is specified. The first issue is that if the alternate cache directory location

Re: [apparmor] [parser patch] fix apparmor cache tempfile location to use passed arg

2013-07-23 Thread John Johansen
On 07/23/2013 08:36 AM, Steve Beattie wrote: Subject: [parser patch] fix apparmor cache tempfile location to use passed arg This patch fixes problems in the handling of both the final cache name location and the temporary cache file when an alternate location is specified. The first issue

Re: [apparmor] [parser patch] fix apparmor cache tempfile location to use passed arg

2013-07-23 Thread Christian Boltz
Hello, Am Dienstag, 23. Juli 2013 schrieb Steve Beattie: The second issue is that it would generate the temporary file that it stores the cache file in [basedir]/cache even if an alternate cache location was specified on the command line. This causes a problem if [basedir]/cache is on a