Re: [PATCH] pfctl configuration file parser

2013-01-25 Thread sven falempin
printf $? On Thu, Jan 24, 2013 at 2:45 AM, Janne Johansson wrote: > Yes, I think the usual "programs should not print if the normal > operation went fine" style should keep pfctl -f some-file quiet. > Having output in the specific config test mode would not hurt, but > it's not something I've bee

Re: [PATCH] pfctl configuration file parser

2013-01-23 Thread Janne Johansson
Yes, I think the usual "programs should not print if the normal operation went fine" style should keep pfctl -f some-file quiet. Having output in the specific config test mode would not hurt, but it's not something I've been longing for. 2013/1/24 Julien Dhaille : > Oops. > This one is better, onl

Re: [PATCH] pfctl configuration file parser

2013-01-23 Thread Julien Dhaille
Oops. This one is better, only display on : # pfctl -nf /etc/pf.conf Cheers Index: pfctl.c === RCS file: /cvs/src/sbin/pfctl/pfctl.c,v retrieving revision 1.314 diff -u -r1.314 pfctl.c --- pfctl.c 19 Sep 2012 15:52:17 -

[PATCH] pfctl configuration file parser

2013-01-23 Thread Julien Dhaille
Hi, Most of daemons like bgpd, ospfd, smtpd, relayd, ... use to return "configuration OK" when the parsing of configuration file is correct. This patch probably seems useless, it's just a thought to stay coherent with an uniform behavior. Index: pfctl.c ===