bin/41820: pppd man page and ip-up and DNS

[reed%reedmedia.net@localhost]

>Number:         41820
>Category:       bin
>Synopsis:       pppd man page and ip-up and DNS
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    bin-bug-people
>State:          open
>Class:          doc-bug
>Submitter-Id:   net
>Arrival-Date:   Wed Aug 05 04:00:00 +0000 2009
>Originator:     reed%reedmedia.net@localhost
>Release:        NetBSD 5.0_STABLE
>Organization:
>Environment:
        
        
System: NetBSD tx.reedmedia.net 5.0_STABLE NetBSD 5.0_STABLE (GENERIC) #0: Mon 
May 11 12:36:19 CDT 2009 
reed%tx.reedmedia.net@localhost:/home/reed/netbsd/src/sys/arch/amd64/compile/GENERIC
 amd64
Architecture: x86_64
Machine: amd64
>Description:
pppd man page says DNS1 and DNS2 are for ip-up script
and then later in same manpage says are for different scripts.

Also description for using the peer DNS is out of order and seems
to imply that the ip-up script is ran before the /etc/ppp/resolv.conf
is created. That is wrong. (I didn't provide patch for that.)
>How-To-Repeat:
        
>Fix:
Move the definition. (Some are alphabetical and some aren't.)
I'd fix this myself, but I see this is in "dist" and I opened this
bug so I can remember.

Index: dist/pppd/pppd/pppd.8
===================================================================
RCS file: /cvsroot/src/dist/pppd/pppd/pppd.8,v
retrieving revision 1.3
diff -u -r1.3 pppd.8
--- dist/pppd/pppd/pppd.8       29 Jun 2006 21:50:17 -0000      1.3
+++ dist/pppd/pppd/pppd.8       5 Aug 2009 03:48:57 -0000
@@ -1568,6 +1568,14 @@
 .B DEVICE
 The name of the serial tty device being used.
 .TP
+.B DNS1
+If the peer supplies DNS server addresses, this variable is set to the
+first DNS server address supplied.
+.TP
+.B DNS2
+If the peer supplies DNS server addresses, this variable is set to the
+second DNS server address supplied.
+.TP
 .B IFNAME
 The name of the network interface being used.
 .TP
@@ -1610,13 +1618,6 @@
 .B LINKNAME
 The logical name of the link, set with the \fIlinkname\fR option.
 .TP
-.B DNS1
-If the peer supplies DNS server addresses, this variable is set to the
-first DNS server address supplied.
-.TP
-.B DNS2
-If the peer supplies DNS server addresses, this variable is set to the
-second DNS server address supplied.
 .P
 Pppd invokes the following scripts, if they exist.  It is not an error
 if they don't exist.

>Unformatted: