I finally got time to test your patch. And it works perfectly! Combined with the previous one, the test case of my first post works as expected. :-)
If some wants to play with it on rsyslog 7.2.6: http://pastebin.com/Y8T7cYS5 Thanks Rainer ! Philippe Muller On Fri, Apr 5, 2013 at 6:44 PM, Rainer Gerhards <[email protected]>wrote: > On Fri, 2013-04-05 at 18:20 +0200, Philippe Muller wrote: > > How should I apply it ? > > > > I tested on 7.2.6, 7.3.9 and git master. It always gets rejected. > > > It probably needs the other patch in v7-stable. So building from git > v7-stable would be one option. I can also see that I merge the patch > up ... even if it were not complete, it fixes something ;) > > Rainer > > Philippe Muller > > > > > > On Thu, Apr 4, 2013 at 4:10 PM, Rainer Gerhards < > [email protected]>wrote: > > > > > Philippe, > > > > > > > FYI: can reproduce, and seems to be present in master branch as well. > > > > > > OK, this was a very weird issue, related to batch processing and state > > > tracking. The then-case worked OK, but the else case was applied > always, > > > even for messages that never were intended to go into the inner if. > > > > > > Bug fix is here: > > > > > > > > > > http://git.adiscon.com/?p=rsyslog.git;a=commitdiff;h=9b85b24d1323c91a06aeef08bbbde7a96afd46d6 > > > > > > It looks like I have covered all cases, but would appreciate if you > could > > > also give it a try. > > > > > > Note that there is some more potential for refactoring; with the patch > > > already being a bit lengthy, I prefer to do that in the development > branch. > > > But first let's see how things evole. > > > > > > Note that while the patch is lengthy, part of it is to improve script > > > debugging output (which is uncritical from a regression PoV). > > > > > > Rainer > > > _______________________________________________ > > > rsyslog mailing list > > > http://lists.adiscon.net/mailman/listinfo/rsyslog > > > http://www.rsyslog.com/professional-services/ > > > What's up with rsyslog? Follow https://twitter.com/rgerhards > > > NOTE WELL: This is a PUBLIC mailing list, posts are ARCHIVED by a > myriad > > > of sites beyond our control. PLEASE UNSUBSCRIBE and DO NOT POST if you > > > DON'T LIKE THAT. > > > > > _______________________________________________ > > rsyslog mailing list > > http://lists.adiscon.net/mailman/listinfo/rsyslog > > http://www.rsyslog.com/professional-services/ > > What's up with rsyslog? Follow https://twitter.com/rgerhards > > NOTE WELL: This is a PUBLIC mailing list, posts are ARCHIVED by a myriad > of sites beyond our control. PLEASE UNSUBSCRIBE and DO NOT POST if you > DON'T LIKE THAT. > > _______________________________________________ > rsyslog mailing list > http://lists.adiscon.net/mailman/listinfo/rsyslog > http://www.rsyslog.com/professional-services/ > What's up with rsyslog? Follow https://twitter.com/rgerhards > NOTE WELL: This is a PUBLIC mailing list, posts are ARCHIVED by a myriad > of sites beyond our control. PLEASE UNSUBSCRIBE and DO NOT POST if you > DON'T LIKE THAT. > _______________________________________________ rsyslog mailing list http://lists.adiscon.net/mailman/listinfo/rsyslog http://www.rsyslog.com/professional-services/ What's up with rsyslog? Follow https://twitter.com/rgerhards NOTE WELL: This is a PUBLIC mailing list, posts are ARCHIVED by a myriad of sites beyond our control. PLEASE UNSUBSCRIBE and DO NOT POST if you DON'T LIKE THAT.
