the --help command line option needs to list all options

NEW
Unassigned

Status

()

12 years ago
9 years ago

People

(Reporter: ray, Unassigned)

Tracking

Trunk
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 2 obsolete attachments)

(Reporter)

Description

12 years ago
It does not. A complete list would come from looking at where the C++ method that handles command-line options gets called and where the javascript function for handling command-line options gets called and listing them out. There are many more than one sees from '--help'.

I got this list together at one point. Will find it again, unless someone has it on hand.

For example, "-P" is listed, but "-profile" is not listed.
(Reporter)

Comment 1

11 years ago
Created attachment 263872 [details] [diff] [review]
adds the -profile flag to the output
Attachment #263872 - Flags: review?(gavin.sharp)
Comment on attachment 263872 [details] [diff] [review]
adds the -profile flag to the output

This patch no longer applies, and it's hard to see what you're actually changing. Can you attach an updated diff -uw, assuming those other changes are whitespace only?
Attachment #263872 - Flags: review?(gavin.sharp)

Comment 3

11 years ago
Created attachment 303540 [details]
Adds and cleans command line arguments for firefox
Attachment #303540 - Flags: ui-review+

Comment 4

10 years ago
 * Please change OS (Bug is for all OS with command line, not only macos)
 * update review flag

Updated

10 years ago
Attachment #303540 - Flags: ui-review+

Updated

10 years ago
Attachment #303540 - Flags: review?(gavin.sharp)
(Reporter)

Comment 5

10 years ago
I think that the 2nd patch is wrong. The -P flag is not the same as the -profile flag. The -P flag takes a name, which is part of the last element of the directory name of the profile directory. It is meant to be displayable in the UI in a user-friendly manner. The -profile flag takes the actual directory, either full path or relative path. The name and the directory are different.

Updated

10 years ago
Attachment #303540 - Attachment is patch: false
Attachment #303540 - Flags: review?(gavin.sharp)

Comment 6

10 years ago
Created attachment 324202 [details] [diff] [review]
Adds and cleans command line arguments for firefox

u8 patch against current cvs
Attachment #303540 - Attachment is obsolete: true
Attachment #324202 - Flags: review?(gavin.sharp)
(Reporter)

Comment 7

10 years ago
This new patch still says "-P" == "-profile", which is not true. Please, if I am wrong, let me know.

Comment 8

10 years ago
Created attachment 324345 [details] [diff] [review]
command line patch

You're right, I've checked and some code comments point out the same thing.
The command line currently makes no difference and just call this 'profile'.
Attachment #324202 - Attachment is obsolete: true
Attachment #324345 - Flags: review?(gavin.sharp)
Attachment #324202 - Flags: review?(gavin.sharp)

Updated

9 years ago
Component: Cmd-line Features → Startup and Profile System
Product: Core → Toolkit
QA Contact: startup

Comment 9

9 years ago
Comment on attachment 324345 [details] [diff] [review]
command line patch

-height and -width are not global options; neither is -inspector. These should not be included in this patch.
Attachment #324345 - Flags: review?(gavin.sharp) → review-
You need to log in before you can comment on or make changes to this bug.