Bug#520696: cfengine2: cfservd fails to parse valid admit rules

2009-05-18 Thread Antonio Radici
Petru Ratiu wrote: If I understood correctly, the patch will make any rule parsable only on the host it's meant for (or if the classes are manually defined on the command line). This prevents me running automated syntax checks to catch some other (valid) syntax errors, because (at least in my set

Bug#520696: cfengine2: cfservd fails to parse valid admit rules

2009-05-18 Thread Petru Ratiu
On Mon, May 18, 2009 at 14:20, Antonio Radici wrote: > > Hi Petru, > actually the parsing will always be done, the patch removes the > InitializeAction() function which causes the problem, all other functions of > parse.c do not have InitializeAction() and that should be the right and > default b

Bug#520696: cfengine2: cfservd fails to parse valid admit rules

2009-05-18 Thread Petru Ratiu
On Mon, May 18, 2009 at 12:29, Antonio Radici wrote: > Petru Ratiu wrote: >> 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 a

Bug#520696: cfengine2: cfservd fails to parse valid admit rules

2009-05-18 Thread Antonio Radici
Petru Ratiu wrote: On Wed, May 13, 2009 at 03:31, Antonio Radici 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.

Bug#520696: cfengine2: cfservd fails to parse valid admit rules

2009-05-18 Thread Petru Ratiu
On Wed, May 13, 2009 at 03:31, Antonio Radici 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, sorr

Bug#520696: cfengine2: cfservd fails to parse valid admit rules

2009-05-12 Thread Antonio Radici
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. I've updated this to git and I've uploaded cfengine2-2.2.9-3 on mentors, can

Bug#520696: cfengine2: cfservd fails to parse valid admit rules

2009-05-12 Thread Antonio Radici
Antonio Radici wrote: Can you confirm this behavior with the *old* version of cfengine? by 'working' I mean that it is doing what the documentation says, in this case allowing hosts from the other IPs to access the file; my supposition is that it should not work even if it does not return the e

Bug#520696: cfengine2: cfservd fails to parse valid admit rules

2009-05-12 Thread Antonio Radici
Petru Ratiu wrote: From what I remember testing when i opened the bug, the "comma-separated" or "one-on-a-line" versions of the config are applied completely on the machines they are meant for (but I'll test again, to be certain). If you want me to test a newer package, say so. If the bug commen

Bug#520696: cfengine2: cfservd fails to parse valid admit rules

2009-05-11 Thread Antonio Radici
Quick update, in case you're wondering :-) I'm still working on this, I'm learnign a good deal of flex and yacc but I think that I'm close to the solution. I will update this bug within this week, sorry for the long delay but this is really nasty! Cheers Antonio -- To UNSUBSCRIBE, email to

Bug#520696: cfengine2: cfservd fails to parse valid admit rules

2009-05-06 Thread Bernd Zeimetz
antonio wrote: > Bernd Zeimetz wrote: >> Hi, >> >> are there any news on this bug? >> Would be really great to have it fixed. >> > > Hi, > sorry for the late update, I will try to have this fixed this weekend. if there's anything I can help you with, please let me know. Thanks, Bernd -- Bern

Bug#520696: cfengine2: cfservd fails to parse valid admit rules

2009-05-06 Thread antonio
Bernd Zeimetz wrote: Hi, are there any news on this bug? Would be really great to have it fixed. Hi, sorry for the late update, I will try to have this fixed this weekend. Cheers Antonio -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe"

Bug#520696: cfengine2: cfservd fails to parse valid admit rules

2009-05-06 Thread Bernd Zeimetz
Hi, are there any news on this bug? Would be really great to have it fixed. Cheers, Bernd -- Bernd Zeimetz Debian GNU/Linux Developer GPG Fingerprint: 06C8 C9A2 EAAD E37E 5B2C BE93 067A AD04 C93B FF79 -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.de

Bug#520696: cfengine2: cfservd fails to parse valid admit rules

2009-03-26 Thread Petru Ratiu
Digging through the list archives I found a better solution, that will not lengthen cfservd.conf: use commas to separate arguments instead of spaces. I edited my problematic cfservd.conf.2 example like this: admit: !linux:: /etc/foo10.0.0,10.0.1 It's parsed properly now, with no e

Bug#520696: cfengine2: cfservd fails to parse valid admit rules

2009-03-23 Thread Petru Ratiu
I should point out that the thread referenced in my bug report died because they suspected some issues with CIDR-style hostmasks. Can someone subscribed to the list provide the extra information needed to reproduce the problem? Thanks. -- "Unix was written to play Spacewar and cheat at Scrabbl

Bug#520696: cfengine2: cfservd fails to parse valid admit rules

2009-03-21 Thread Petru Ratiu
Package: cfengine2 Version: 2.2.8-1 Severity: normal *** Please type your report below this line *** When parsing the admit: section of cfservd.conf, cfservd declares invalid a declaration found within an undefined class if it contains more than one hostmask (or a hostmask and another root= or en