Closed
Bug 102766
Opened 23 years ago
Closed 23 years ago
Please move localizable mapi files to en-win.jar
Categories
(MailNews Core :: Simple MAPI, defect, P4)
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9.8
People
(Reporter: tao, Assigned: srilatha)
References
Details
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
sspitzer
:
superreview+
|
Details | Diff | Splinter Review |
Looking at the MAPI_SUPP_BRANCH, I found that there are two localizable files,
"mapi.properties" and "pref-mailnewsOverlay.dtd", in
"mozilla/mailnews/mapi/resources/locale/en-US", and will be packaged into
en-US.jar!chrome://messenger/locale/.
If MAPI is Windows only feature, the UI strings should live in en-win.jar
instead of en-US.jar.
Updated•23 years ago
|
Priority: -- → P4
Comment 1•23 years ago
|
||
Pls "nsbranch-", if we are not gonna fix this one in the branch.
Keywords: nsbranch
Comment 2•23 years ago
|
||
This issue has been resolved through email discussions. Copied the emails here.
(will fix after 0.9.4.)
Srilatha Moturi wrote:
Ok then. I will not move them now.
Srilatha
Michele Carlson wrote:
Yes - I agree with Ray. Please do not move files at this point.
thanks.
Michele
Ray Chen wrote:
There could be a potential problem for L10N. Moving the files around would cause
the file path mismatch and could not be leveraged properly. At this moment (UI
freeze already), it is better not to change it. Since these two files are so
small (about 10 strings), leaving them in en-us.jar won't harm. You can fix it
after 0.9.4.
Thanks,
- Ray
Tao Cheng wrote:
Since there is no impact on the regus.xpi (US.jar), I am fine with the change.
It's L10n group's call whether we accept moving files from en-US.jar to en-win.jar.
As to moving files from en-US.jar to en-win.jar, see my comment below.
Michele Carlson wrote:
If the files have been in the branch since 9/17 we should be fine from a l10n
point of view. The UI has already been translated.
thanks
Michele
Srilatha Moturi wrote:
These files are on the 0.9.4 branch for a while now (checked in on 9/17).
I did not know about en-win.jar. I can make the change.The changes will be in
jar.mn file so I'm guessing this should not effect localization at all.
In addition to the jar.mn change, you'd need to
update the chrome urls referencing the dtd and property file from
chrome://messenger/locale/* to chrome:://messenger -platform/locale/*
create a new package, "messenger-platform"
update makefiles and installer script to reflect the changes in chrome registry.
make sure that the en-win.jar on Mac and Linux contains the MAPI files.
QA the change....
-
Tao
Assignee | ||
Updated•23 years ago
|
Comment 3•23 years ago
|
||
Mark as depedent to meta bug 103807
Assignee | ||
Comment 4•23 years ago
|
||
Updated•23 years ago
|
Target Milestone: --- → mozilla0.9.7
Assignee | ||
Comment 5•23 years ago
|
||
Changed the locale version to 0.9.6
Attachment #54748 -
Attachment is obsolete: true
Assignee | ||
Comment 6•23 years ago
|
||
ccing Seth and Bhuvan for sr and r
Comment 8•23 years ago
|
||
Comment on attachment 57663 [details] [diff] [review]
updated the version number
seems ok to me.
do you need to make similar changes to the netscape commercial tree?
Attachment #57663 -
Flags: superreview+
Assignee | ||
Comment 9•23 years ago
|
||
No, Changes in the commercial tree are not necessary because there is no
mapi/locale directory in the ns tree and also there is not need to override
contents.rdf (since there is no existing contents.rdf in the mailnews directory
for messenger-platform)
Reporter | ||
Comment 10•23 years ago
|
||
Since there was no messenger-platform package before, you want to register it in
makefiles so the new package could be recognized in development builds. Same things
apply to installer scripts so commercial builds work as expected.
Updated•23 years ago
|
Target Milestone: mozilla0.9.7 → mozilla0.9.8
Updated•23 years ago
|
Attachment #57663 -
Flags: review+
Assignee | ||
Comment 11•23 years ago
|
||
Updated the patch against the trunk. Since messenger-mapi package was already
created, using that instead of creating "messenger-platform".
Comment 12•23 years ago
|
||
is that last patch complete? second patch has other changes.
Assignee | ||
Comment 13•23 years ago
|
||
Yes it is complete, since the contents.rdf for messenger-mapi, changes in
pref-mailnewsOverlay.xul and jar.mn already exist in the trunk. These were
checked in by mscott. see bug # 112724
Comment 14•23 years ago
|
||
r=racham.
Updated•23 years ago
|
Attachment #57663 -
Attachment is obsolete: true
Comment 15•23 years ago
|
||
Comment on attachment 64397 [details] [diff] [review]
updated patch against the trunk
sr=sspitzer
Attachment #64397 -
Flags: superreview+
Assignee | ||
Comment 16•23 years ago
|
||
Fix checked in
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•