[apparmor] [patch] utils: fix python install for rule/ subdirectory

2015-01-13 Thread Steve Beattie
utils: fix python install for rule/ subdirectory For reasons that are unclear to me, python's setuptools doesn't install recursively from a directory, meaning that on make install, the new Rules/Ruleset classes were not being installed. This patch causes the rule subdirectory to be included.

Re: [apparmor] [patch] utils: fix python install for rule/ subdirectory

2015-01-13 Thread Steve Beattie
On Tue, Jan 13, 2015 at 01:43:33PM -0600, Tyler Hicks wrote: On 2015-01-13 11:33:10, Steve Beattie wrote: utils: fix python install for rule/ subdirectory For reasons that are unclear to me, python's setuptools doesn't install recursively from a directory, meaning that on make install,

Re: [apparmor] [patch] utils: fix python install for rule/ subdirectory

2015-01-13 Thread Tyler Hicks
On 2015-01-13 11:33:10, Steve Beattie wrote: utils: fix python install for rule/ subdirectory For reasons that are unclear to me, python's setuptools doesn't install recursively from a directory, meaning that on make install, the new Rules/Ruleset classes were not being installed. This patch

Re: [apparmor] [patch] utils: fix python install for rule/ subdirectory

2015-01-13 Thread Christian Boltz
Hello, Am Dienstag, 13. Januar 2015 schrieb Steve Beattie: Ah right, I wrote imprecisely in my patch header; we're actually using python's included distutils and not setuptools, which is a separate project, so don't have access to find_packages(). I'd meant to offer converting to setuptools

Re: [apparmor] [patch] rule class - split out common parts from is_covered()

2015-01-13 Thread Steve Beattie
On Fri, Dec 19, 2014 at 12:24:02AM +0100, Christian Boltz wrote: Hello, this patch splits is_covered() in capability.py into - is_covered_localparts() for rule-specific code - is_covered() for common code - located in __init__.py I'm not keen on the naming convention x_localvars() that