it seems that we have lots of config files that currently have no
guidelines as to what needs quoting, and what does not. the diff below
is for pf.conf.5, and attempts to clarify that.

if no one disagrees, i'll do it for all those that use parse.y. that
is:

        /usr/src/sbin/iked/parse.y
        /usr/src/sbin/ipsecctl/parse.y
        /usr/src/sbin/pfctl/parse.y
        /usr/src/usr.sbin/bgpd/parse.y
        /usr/src/usr.sbin/dvmrpd/parse.y
        /usr/src/usr.sbin/hostapd/parse.y
        /usr/src/usr.sbin/ifstated/parse.y
        /usr/src/usr.sbin/iscsictl/parse.y
        /usr/src/usr.sbin/ldapd/parse.y
        /usr/src/usr.sbin/ldpd/parse.y
        /usr/src/usr.sbin/mrouted/cfparse.y
        /usr/src/usr.sbin/ntpd/parse.y
        /usr/src/usr.sbin/ospf6d/parse.y
        /usr/src/usr.sbin/ospfd/parse.y
        /usr/src/usr.sbin/relayd/parse.y
        /usr/src/usr.sbin/ripd/parse.y
        /usr/src/usr.sbin/smtpd/parse.y
        /usr/src/usr.sbin/snmpd/parse.y
        /usr/src/usr.sbin/ypldap/parse.y

any dissention, or inaccuracies?

jmc

Index: pf.conf.5
===================================================================
RCS file: /cvs/src/share/man/man5/pf.conf.5,v
retrieving revision 1.513
diff -u -r1.513 pf.conf.5
--- pf.conf.5   31 Jan 2012 07:46:32 -0000      1.513
+++ pf.conf.5   21 Apr 2012 17:32:13 -0000
@@ -72,14 +72,17 @@
 .Pp
 Additional configuration files can be included with the
 .Ic include
-keyword, for example:
+keyword.
+Note that argument names not beginning with a letter, digit, or underscore
+must be quoted.
+For example:
 .Bd -literal -offset indent
 include "/etc/pf/sub.filter.conf"
 .Ed
 .Pp
 Macros can be defined that will later be expanded in context.
-Macro names must start with a letter, and may contain letters, digits
-and underscores.
+Macro names must start with a letter, digit, or underscore,
+and may contain any of those characters.
 Macro names may not be reserved words (for example
 .Ar pass ,
 .Ar in ,

Reply via email to