Errors in usage string

RESOLVED WORKSFORME

Status

()

Firefox
Installer
--
minor
RESOLVED WORKSFORME
12 years ago
9 years ago

People

(Reporter: Hendrik Maryns, Unassigned)

Tracking

1.5.0.x Branch
x86
Windows XP
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [expired])

Attachments

(1 attachment)

(Reporter)

Description

12 years ago
In trying to localise the string MSG_USAGE, found in browser/installer/installer.inc, we got convinced that there is an error in there.

It says
#define MSG_USAGE [...] -greShared: Forces GRE to be installed into a global, shared dir (normally)\n			c:\program files\common files\mozilla.org\GRE\n

Here the closing parenthesis after normally should come at the end, between GRE and the newline sign.  And when we are there, I very much doubt the correctness of the given path.  Isn’t this file XP?  And Program Files is written with capitals, usually (I know it is insensitive, but anyway).  And isn’t a period missing, too?

[...]  -greForce: Force 'Component GRE' to be downloaded, run, and installed.  This\n			bypasses GRE's logic of determining when to install by running its\n			installer with a '-f' flag.\n

Shouldn’t that be ‘an '-f' flag’?

[...]  -cleanupOnUpgrade: Tells Setup to check to see if user is upgrading (installing on top\n		of previous version of product).  If user is upgrading:\n			* NORMAL mode: prompt user on how to proceed\n			* All other modes: assume user wants to cleanup.\n	*

Here it is unclear what NORMAL mode is.  As localisers, we have no idea whether we should localise this word, leave it in caps, ...?  And is it the modes that assume, or is this assume an infinitive (not an error, but unclear, another debate in #mozilla.nl)

[...] -noCleanupOnUpgrade: Tells Setup to not check if user is upgrading (installing on top\n		of previous version of product).  This will disable the cleanup feature.\n\n	* means it will override config.ini

Either ‘means’ deserves a capital (it follows a period), or something else is wrong in this sentence.
(Reporter)

Comment 1

12 years ago
Created attachment 206241 [details] [diff] [review]
Proposes some of the changes, incomplete

There are some more problems: 
- what are all the asterisks for?
- Does the last line follow the last option or is it a general remark (I suppose so)?
I added some periods too, but not enough, I think.
(Reporter)

Comment 2

12 years ago
(In reply to comment #1)
> There are some more problems: 
> - what are all the asterisks for?
> - Does the last line follow the last option or is it a general remark (I
> suppose so)?

In the meantime I got some clarification about this on IRC, but the fact that I read this sentence a dozen times without understanding seems to signal some clarity is needed.  How about adding an article: ‘A * means it will override config.ini.’
Changing to 1.5.0.x branch since Firefox is now using a new installer
Version: Trunk → 1.5.0.x Branch
1.5 Branch is no longer supported, I'm gonna close this as WORKSFORME, please reopen if the problem still exists on a currently supported branch.
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → WORKSFORME
Whiteboard: [expired]
You need to log in before you can comment on or make changes to this bug.