Merge lp:~gleichsnerd/ubuntu/precise/mountall/fix-for-805509 into lp:ubuntu/precise/mountall

[Adam Gleichsner]

As requested in #805509, I added the options for mountall to the man page, as written in mountall --help.

Testing is straightforward, just backup the old mountall man page and throw in this one. Everything should come out all squeaky clean and beautiful.

**5/29/12 -- Formatting updated/fixed; resubmitted

[Micah Gersten (micahg)]

This looks great, thanks. Are you ok with this patch being under Canonical copyright?

review: Needs Information

[James Hunt (jamesodhunt)]

Hi Adam - thank you very much for your contribution. A few minor comments:

- All dashes ('-') should be escaped so rather than '--debug', you should specify '\-\-debug'.
- '--dev-wait-time=value' should be '\-\-dev\-wait\-time=\fIvalue\fP'.
- 'value' in '--dev-wait-time' is not underlined.
- Formatting is wrong for '--dev-wait-time'.
- '--quiet' and '--verbose' are not displaying in bold as the other options are.

Take a look at the init-checkconf(8) man page as this has examples of all the above scenarios:

    vim $(man -w init-checkconf)

[Adam Gleichsner (gleichsnerd)]

Sorry for the hiatus, I had some hardware malfunctions that put me out of commission for awhile.

I fixed the formatting as per James' request. Hopefully it's all set to go now.

Let me know if anything else needs to be done.

-Adam

P.S. @Micah - The Canonical copyright is fine.

review: Needs Resubmitting

[James Hunt (jamesodhunt)]

Hi Adam,

Thanks very much for your work on this - looks great now!

(BTW - if you're interested, we don't yet have a man page for /sbin/mntctl in the mountall package).

[Didier Roche-Tolomelli (didrocks)]

Merged in lp:ubuntu/mountall trunk after some discussion with jodh. Thanks! (this branch will be rejected as wrong target)

review: Approve

[Adam Gleichsner (gleichsnerd)]

@James Hunt,

I would be happy to. I just submitted a bug report to link to it; I'll get cracking right away.

Unmerged revisions

383. By Adam Gleichsner on 2012-05-30

Fixed formatting errors

382. By Adam Gleichsner on 2012-03-31

Added options to mountall(8) man page

=== modified file 'man/mountall.8'
--- man/mountall.8	2009-09-16 22:38:18 +0000
+++ man/mountall.8	2012-05-30 03:17:18 +0000
@@ -25,6 +25,39 @@
 itself gains the necessary flexibility to perform this processing; you
 should not rely on its behaviour.
 .\"
+.SH OPTIONS
+
+.BR      \-\-daemon
+\               Detach and run in the background
+
+.BR      \-\-force-fsck
+\           Force check of all filesystems
+
+.BR      \-\-fsck-fix
+\             Attempt to fix all fsck errors
+
+.BR      \-\-no-events
+\            Do not emit events after mounting filesystems
+
+.BR      \-\-dev\-wait\-time=\fIvalue\fP "   (in seconds)"
+                        (Default is 30 seconds, Legal value 
+                        is between 1 to 2147483647)
+                        In case of (bootwait,timeout): specify
+                        the time to wait for device to be detected
+
+.BR      \-q " , " \-\-quiet
+\           Reduce output to errors only
+
+.BR      \-v " , " \-\-verbose
+\         Increase output to include informational
+                        messages
+
+.BR      \-\-help
+\                 Display this help and exit
+
+.BR      \-\-version
+\              Output version information and exit
+.\"
 .SH AUTHOR
 Written by Scott James Remnant
 .RB < [e-mail] >