[apparmor] [patch 3/3] use capability rule class in aa.py and cleanprof.py

[Seth Arnold]

On Sun, Nov 30, 2014 at 12:45:49AM +0100, Christian Boltz wrote:
> > Second is that I think these changes are large enough to not be
> > acceptable for 2.9.1, and that we should branch off 2.9.x before
> > committing this patch set.

> You are right that the changes are quite big for a maintenance release. 
> 
> However there are some reasons that let me tend to include it in 2.9.x:
> - as a side effect of the conversation to classes, we fix some bugs. 
>   Maybe we even accidently ;-) fix some bugs we didn't even notice 
>   before.
> - the classes are fully covered by tests, which means the risk for 
>   regressions is quite small (we can "only" break aa.py)
> - we'll have a separate branch that we need to support with bugfixes if
>   we don't include the rule classes in 2.9.x ;-)

I think I'd rather see "all these" (loosely used to refer to all the
work you guys have put in the last few weeks) rolled into trunk before
branching 2.9.x. These changes are rather more than the usual bug-fixes,
true, but they are addressing the usability issues with the current code.

I'm afraid having a trunk and a 2.9 branch would blunt the current
enthusiasm that appears to be very fruitful recently.

Thanks
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20141201/88db1289/attachment.pgp>