RE: rfv: stop looking for config dirs when done

2007-07-04 Thread Bruce Shaw
+1 On ons, 2007-07-04 at 15:25 -0400, Robert Story wrote: > By sheer chance, I found a bug in read_config. I was checking to see what > files an agent was reading, and saw lots of 'directory not present' messages > logged for weird name (some including control characters). A short debugging > s

Re: rfv: stop looking for config dirs when done

2007-07-04 Thread Thomas Anders
Robert Story wrote: > By sheer chance, I found a bug in read_config. I was checking to see what > files an agent was reading, and saw lots of 'directory not present' messages > logged for weird name (some including control characters). A short debugging > session later, I found that if the last dir

Re: rfv: stop looking for config dirs when done

2007-07-04 Thread Marcus Rueckert
On 2007-07-04 15:25:01 -0400, Robert Story wrote: > By sheer chance, I found a bug in read_config. I was checking to see what > files an agent was reading, and saw lots of 'directory not present' messages > logged for weird name (some including control characters). A short debugging > session later

Re: rfv: stop looking for config dirs when done

2007-07-04 Thread Magnus Fromreide
On ons, 2007-07-04 at 15:25 -0400, Robert Story wrote: > By sheer chance, I found a bug in read_config. I was checking to see what > files an agent was reading, and saw lots of 'directory not present' messages > logged for weird name (some including control characters). A short debugging > session

Re: rfv: stop looking for config dirs when done

2007-07-04 Thread Dave Shield
On 04/07/07, Robert Story <[EMAIL PROTECTED]> wrote: > This patch adds an extra condition to the while check, instead of > relying on a break inside the loop. Seems sensible (and safe). +1 Dave - This SF.net email is sponso

rfv: stop looking for config dirs when done

2007-07-04 Thread Robert Story
By sheer chance, I found a bug in read_config. I was checking to see what files an agent was reading, and saw lots of 'directory not present' messages logged for weird name (some including control characters). A short debugging session later, I found that if the last directory in the config list do