Closed
Bug 572221
Opened 14 years ago
Closed 14 years ago
Remove call to check-locales.pl for SeaMonkey
Categories
(SeaMonkey :: Installer, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)
References
Details
Attachments
(1 file)
2.02 KB,
patch
|
Callek
:
review+
kairo
:
feedback+
|
Details | Diff | Splinter Review |
Spinoff of bug Bug 570680 for SeaMonkey This was added to let localizers know that they needed to change ${BrandShortName} to $BrandShortName and ${BrandFullName} to $BrandFullName instead of changing the string names. check-locales.pl won't be removed until all apps using it no longer need to use it.
Assignee | ||
Comment 1•14 years ago
|
||
Since this was added (fixed actually) on on 2009-01-13 in Bug 470392 I think it is safe to remove now
Assignee | ||
Comment 2•14 years ago
|
||
Assignee: installer → robert.bugzilla
Status: NEW → ASSIGNED
Attachment #451535 -
Flags: review?(bugspam.Callek)
Comment 3•14 years ago
|
||
I'll review later tonight; But since KaiRo is up now, I figured i'll feedback? him too. [Localization stuff he likes to know about]
Comment 4•14 years ago
|
||
Comment on attachment 451535 [details] [diff] [review] patch Actually simpler than I expected.
Attachment #451535 -
Flags: review?(bugspam.Callek)
Attachment #451535 -
Flags: review+
Attachment #451535 -
Flags: feedback?(kairo)
Updated•14 years ago
|
Attachment #451535 -
Flags: feedback?(kairo) → feedback+
Assignee | ||
Comment 5•14 years ago
|
||
Pushed to comm-central http://hg.mozilla.org/comm-central/rev/05dcd2dd352d
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•