431636 - "seamonkey -h" should mention -migration switch

Status: RESOLVED FIXED

(SeaMonkey :: Help Documentation, defect)

Description

[Tony Mechelynck [:tonymec]]

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9pre) Gecko/2008043002 SeaMonkey/2.0a1pre

Reproducible: Always

Steps to Reproduce:
1. (at shell prompt) seamonkey -h

Actual Results:
-migration switch is not mentioned

Expected Results
-migration switch (to import profile data from other Netscape/Mozilla apps incl. Sm 1.x) should be mentioned.

Additional info:
See also bug 383006

Comment 2

[Ian Neal]

Attachment: Tweak -h information

This patch:
* Adds -P <profile> -migration to app options
* Changes "Mozilla options" to app options
* Removes \t which causes one line not to line up

Comment 3

[[e-mail]]

Comment on attachment 334582 [details] [diff] [review]
Tweak -h information

>-         "\t-UILocale <locale>\t\tStart with <locale> resources as UI Locale.\n"
>+         "\t-UILocale <locale>\tStart with <locale> resources as UI Locale.\n"
Maybe these should just use spaces. By comparison
  -jsconsole           Open the Error console.
  -reftest <file>    Run layout acceptance tests on given manifest.
Hmm, well I guess we aren't as consistent as we should be ;-)

Comment 4

[Benjamin Smedberg]

Comment on attachment 334582 [details] [diff] [review]
Tweak -h information

the -migration flag does not have to be used in conjunction with -P. It should be documented separately.

Comment 5

[Ian Neal]

Attachment: Revised tweak to -h information v0.1a (Checkin: Comment 8)

Changes since v0.1:
* Now documents -migration separate to -P <profile>

Alternative is to say:
"Start with migration wizard, migrates into default profile if none is specified."
but that might be a bit too long.

Comment 6

[Ian Neal]

Not something we need to get fixed for SM2.0a1 but will take a fix if there is one.

Comment 7

[Robert Kaiser]

Comment on attachment 335409 [details] [diff] [review]
Revised tweak to -h information v0.1a (Checkin: Comment 8)

>diff --git a/toolkit/xre/nsAppRunner.cpp b/toolkit/xre/nsAppRunner.cpp
> #ifdef XP_UNIX
>   printf("\t--g-fatal-warnings\t\tMake all warnings fatal\n"
>-         "\nMozilla options\n");
>+         "\n%s options\n", gAppData->name);

Sorry for nitpicking, but wouldn't it be better to make the second line |#ifdef MOZ_X11| just line the other such subsection heading is?

Comment 8

[Ian Neal]

Comment on attachment 335409 [details] [diff] [review]
Revised tweak to -h information v0.1a (Checkin: Comment 8)

http://hg.mozilla.org/mozilla-central/rev/0e80ec86dc1d