Bug 84408

Summary: [patch] dump(8) manpage doesn't require an option and has wrong default
Product: Documentation Reporter: Gary W. Swearingen <garys>
Component: Books & ArticlesAssignee: ru <ru>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
file.diff none

Description Gary W. Swearingen 2005-07-31 19:30:22 UTC
-- dump(8) manpage synopsis doesn't require use of an option like the
program does. (An option is not optional!)

    $ dump /dev/ad0s3e
    dump: option requires an argument -- d

-- It also lists the wrong default /dev/sa0:

    $ dump -a /dev/ad0s3e
    ...
    DUMP: Cannot open output "/dev/nsa0".

-- Freebie: dump isn't restricted to just a tape unit so it doesn't
   only have a "default tape unit". I changed wording.

How-To-Repeat: n/a
Comment 1 Gary W. Swearingen 2005-08-01 00:35:29 UTC
I was wrong about the "wrong default"; I forgot I had TAPE defined
when I tested.

The patch should have only been:

--- dump..orig.8	Sun Jul 31 11:11:22 2005
+++ dump.8	Sun Jul 31 11:18:27 2005
@@ -38,7 +38,7 @@
 .Nd file system backup
 .Sh SYNOPSIS
 .Nm
-.Op Fl 0123456789acLnSu
+.Fl 0123456789acLnSu
 .Op Fl B Ar records
 .Op Fl b Ar blocksize
 .Op Fl C Ar cachesize
Comment 2 dfilter service freebsd_committer freebsd_triage 2006-10-12 21:22:39 UTC
ru          2006-10-12 20:22:31 UTC

  FreeBSD src repository

  Modified files:
    sbin/dump            main.c 
  Log:
  Make "dump /filesystem" (without options) work.
  
  PR:             docs/84408
  MFC after:      3 days
  
  Revision  Changes    Path
  1.65      +5 -2      src/sbin/dump/main.c
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 3 ru freebsd_committer freebsd_triage 2006-10-12 21:22:49 UTC
State Changed
From-To: open->closed

Actually they are optional; the old BSD 4.3 suntax support got in 
the way (should be fixed now). 


Comment 4 ru freebsd_committer freebsd_triage 2006-10-12 21:22:49 UTC
Responsible Changed
From-To: freebsd-doc->ru
Comment 5 dfilter service freebsd_committer freebsd_triage 2006-10-16 12:52:17 UTC
ru          2006-10-16 11:52:00 UTC

  FreeBSD src repository

  Modified files:        (Branch: RELENG_6)
    sbin/dump            main.c 
  Log:
  MFC: 1.65: Make "dump /filesystem" (without options) work.
  
  PR:             docs/84408
  Approved by:    re (hrs)
  
  Revision  Changes    Path
  1.63.2.2  +5 -2      src/sbin/dump/main.c
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"