Closed Bug 207280 Opened 21 years ago Closed 17 years ago

several -help entries missing

Categories

(Core Graveyard :: Cmd-line Features, defect)

x86
All
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: bugzilla, Unassigned)

References

Details

(Keywords: regression)

Attachments

(1 file, 2 obsolete files)

found on linux rh8.0. not a problem on Mac OS X; unable to check win2k due to bug 26761. when i issue the -help option, the listing should include tips on using composer, mail and so forth (eg, -edit, -mail). sometime btwn the 1/13 and 1/31 builds, these options are no longer listed --it's as if the listing stopped after the -splash option. when it did work (and i'll narrow down the regression window in a bit), here's what the -help list (the portion after the -splash entry) included (commercial): -splash Enable splash screen. -edit <url> Start with editor. -addressbook Start with the addressbook. -news Start with news. -jsconsole Start with Javascript Console -aim Start with Instant Messenger -chrome <url> Load the specified chrome. -mail Start with mail. -compose <url> Start with messenger compose.
looking at commercial linux builds, this regressed sometime btwn 2003.01.16.08 (works) and 2003.01.21.10 (broken). (unfortunately, i couldn't find other builds btwn those dates to test.)
seth confirmed that this also occurs in his win32 debug. strange that this is not a problem on mac os x...
OS: Linux → All
Summary: Linux: several -help entries missing → several -help entries missing
I'm not sure if ssu is the right owner, but I'm sure he's more than capable of fixing this. odd that it works on mac.
use "mozilla.exe -help |more" on win32 in a non-debug build. An older working copy from the help output: http://www.mversen.de/mozilla/commands.txt
thanks, Matti! here's the output from 2003.05.23.09 comm build on win2k; note how entries after -quiet are missing. $ ./netscp -help |more Usage: c:\Test Builds\2003052309\netscp.exe [ options ... ] [URL] where options include: -height <value> Set height of startup window to <value>. -h or -help Print this message. -installer Start with 4.x migration window. -width <value> Set width of startup window to <value>. -v or -version Print c:\Test Builds\2003052309\netscp.exe versi on. -CreateProfile <profile> Create <profile>. -P <profile> Start with <profile>. -ProfileWizard Start with profile wizard. -ProfileManager Start with profile manager. -SelectProfile Start with profile selection dialog. -UILocale <locale> Start with <locale> resources as UI Loca le. -contentLocale <locale> Start with <locale> resources as content Locale. -console Start Mozilla with a debugging console. -nosplash Disable splash screen. -quiet Disable splash screen.
Attached patch Pach (obsolete) (deleted) — Splinter Review
Comment on attachment 129633 [details] [diff] [review] Pach > // this works, but only after the components have registered. so if you drop in a new command line handler, -help > // won't not until the second run. > // out of the bug, because we ship a component.reg file, it works correctly. > DumpArbitraryHelp(); It looks to me as this is what has stopped working.
Attachment #129633 - Flags: review-
Attached patch New patch (obsolete) (deleted) — Splinter Review
Attachment #129633 - Attachment is obsolete: true
Attachment #135888 - Flags: review?(neil.parkwaycc.co.uk)
Comment on attachment 135888 [details] [diff] [review] New patch 1. biesi asks that you use -p when making diffs 2. i ask that you not break -help on unix, which your patch does. Here's how to verify that it breaks: unset DISPLAY run ./mozilla -help Expected results: help actual results based on my understanding of the code and biesi's testing: crash near: > 1591 gtk_widget_set_default_visual(gdk_rgb_get_visual()); > 1592 gtk_widget_set_default_colormap(gdk_rgb_get_cmap()); You need to make sure that help does not happen after any gui/splashy things, which this patch unfortunately changes. I'm traveling, but I'll be able to access a computer a few times a week for the next two months, so I would like to be able to veto whatever patch you make for this bug. Note that we're currently in a freeze for 1.6b, and I respectfully request that any driver considering any change to startup specifically for this bug reject it as risky. Here are some reasons: 1. it's a change to startup code, which is inherently risky 2. startup code is very fragile. 3. the person who seems to be writing this patch at the present is not aware of the many ways that this code can break or the reasons that it grew the way it did.
Attachment #135888 - Flags: review?(neil.parkwaycc.co.uk) → review-
.
Assignee: ssu → nobody
Attached patch Patch again (deleted) — Splinter Review
OK, I just want to init XPCOM before "HandleDumpArguments". If I can not move this line down, I can only init XPCOM early.
Attachment #135888 - Attachment is obsolete: true
Attachment #136212 - Flags: review?(timeless)
Comment on attachment 136212 [details] [diff] [review] Patch again offhand 'no' and i'm supposed to be packing because my train leaves in less than 6hrs. you'll mess up the profiling init among other stuff. please just leave this problem unsolved for the time being.
Attachment #136212 - Flags: review?(timeless) → review-
Keywords: relnote
*** This bug has been marked as a duplicate of 355889 ***
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → DUPLICATE
comment 0 says "found on linux" while that bug is windows-specific
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
When SeaMonkey switches to toolkit, this should be fixed (test builds already show all the options again), therefore adding a dep on bug 328887).
Depends on: suiterunner
This now works again on SeaMonkey trunk, mainly due to the switch over to xul app in bug 328887. Several bugs probably fixed this, so marking as WFM.
Status: REOPENED → RESOLVED
Closed: 18 years ago17 years ago
Resolution: --- → WORKSFORME
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: