On Sun, Sep 21, 2014 at 04:04:48AM -0700, John Johansen wrote: > On 09/19/2014 06:48 PM, Seth Arnold wrote: > > On Fri, Sep 19, 2014 at 01:48:09PM -0700, John Johansen wrote: > >> On 09/19/2014 12:37 PM, John Johansen wrote: > >> > >> fix: if the apparmor parser fails to load the cache try rebuilding > >> > >> v2. Change bail out condition, it should skip rebuilding the cache if the > >> load succeed > >> > >> Signed-off-by: John Johansen <john.johan...@canonical.com> > > > > Some quibbles on this one... First, the variable name makes a lot more > > sense to me than the command line option -- I'd rather the command line > > option match, "skip-rebuild-bad-cache" or "skip-rebuild-on-bad-cache". > > Second, the --help isn't updated yet. > > > > But I think the logic works alright.
The logic looks good to me, too, and experimentally works as expected. > How's this > > fix: if the apparmor parser fails to load the cache try rebuilding > > Signed-off-by: John Johansen <john.johan...@canonical.com> Acked-by: Steve Beattie <st...@nxnw.org>, thanks. -- Steve Beattie <sbeat...@ubuntu.com> http://NxNW.org/~steve/
signature.asc
Description: Digital signature
-- AppArmor mailing list AppArmor@lists.ubuntu.com Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
