specifying -O to parser causes the parser to not consider cache files

So this is working as designed, when you explicitly override the Optimization flags it is assumed that you will not want the old compile.

There are a couple of ways to properly fix this
1. Store the compile options used with the cache, and compare them at the start of a compile, to determine whether or not to skip using the cache.

2. add a new flag to indicate that the implied skip cache shouldn't be used. This is easier but some what hacky.

A third option is open to Ubuntu, to use a distro patch (I don't think this will be taken upstream, at least not with out discussion) to remove forcing optimization options to skip the cache. This is the easiest and the patch to do so is attached

The attachment "foo.diff" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

Thanks John. For now I'll use this patch and we can discuss if we want to do something different upstream as we have time.

In thinking about this a little, it seems that if optimization flags can be set in parser.conf to change the defaults, then use of '-O' shouldn't force skipping reading of the cache. Perhaps it would make sense to apply this patch but also to adjust the man page to state something long the lines of "if specifying '-O' on the command line, it probably makes sense to also specify '--skip-read-cache'".

Sticking it the parser.conf also counts as overriding because the set of flags used during compile are currently not stored (basically anything different than the default binary behavior is treated as different).

The proper long term fix is to store this information off so that the parser can detect compile option changes.

This bug was fixed in the package apparmor - 2.8.98-0ubuntu3

---------------
apparmor (2.8.98-0ubuntu3) vivid; urgency=medium

  * debian/lib/apparmor/functions: disable expr tree simplification for
    /var/lib/apparmor/profiles (LP: #1383858)
  * parser-dont-skip-read-cache-with-optimizations.patch: don't skip read
    cache when specifying '-O' (LP: #1385947)
 -- Jamie Strandboge <email address hidden> Tue, 28 Oct 2014 17:41:08 -0500

This bug was fixed in the package apparmor - 2.8.96~2652-0ubuntu5.3

---------------
apparmor (2.8.96~2652-0ubuntu5.3) 14.09; urgency=medium

  * debian/lib/apparmor/functions: disable expr tree simplification for
    /var/lib/apparmor/profiles (LP: #1383858)
  * parser-dont-skip-read-cache-with-optimizations.patch: don't skip read
    cache when specifying '-O' (LP: #1385947)
  * 10-lp1371771.patch, 11-lp1371765.patch: refresh for the above
 -- Jamie Strandboge <email address hidden> Mon, 27 Oct 2014 07:13:42 -0500