Petru Ratiu wrote:
On Wed, May 13, 2009 at 03:31, Antonio Radici <anto...@dyne.org> wrote:
tag 520696 -moreinfo
tag 520696 patch pending
thanks
Ok, the patch is attached, it is very simple :-)
It will just skip the entire action when the class does not exist, same as
the other Install* functions are doing.
Hello, sorry for the delay.
I haven't yet tested the patch, but I find it not quite right, because
I still want to be able to parse and syntax-check the configs (I have
a svn hook that does syntax checking and this will cause it to neglect
real syntax errors for rules not meant for the machine doing the
check).
I'll try to test the package as soon as I can, but I need to say that
I won't use this fix in production and would prefer the current
behavior.
Probably there was a misunderstanding, having two addresses separated by
a space is not a syntax error, the only reason why this is reported as
syntax error is because of this bug; the patch is not removing any
syntax check.
The patch is solving the problem of wrong syntax check failures when the
class does not exist and admit/grant directives are separated by one
space; if the same directives are included in an existing class no
syntax error will be reported.
I hope this clarifies the context.
Cheers
Antonio
--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
