14379 - [L12y] PLS remove locale name (en-US) from the localMsgs.properties

Status: VERIFIED FIXED

(MailNews Core :: Backend, defect, P3)

Description

[tao]

Hi, Scott:

This is probably a residue from our out-dated localizability strategy.
Now, we plan to treat property files as *.dtd and place them under
locale/[locale-name] directories. Therefore, we do not need to embed
locale name in the filename anymore.

PLS bounce back to me if you'd like me to fix it. Let's check this in
today.

Thanks.

Comment 1

[tao]

Oops, here is the target file:

http://lxr.mozilla.org/mozilla/source/mailnews/local/resources/locale/en-US/localMsgs_en-US.properties

Comment 2

[Scott MacGregor]

So we have a new plan for this eh? =)

So localMsgs.properties should be the name of the file and it stays in the same
directory?

If you have cycles to do this today then I'll re-assign this to you as it would
get done faster that way. I probably won't get to it until after M11 tree
closes.

Comment 3

[tao]

Yes, right and the same directory, too.

I am taking it over. But, would you kindly let me know where it is used; just
in case that I miss one.

Thanks

Comment 4

[tao]

One more

/mailnews/compose/resources/locale/en-US/composeMsgs_en.properties

Comment 5

[Scott MacGregor]

there's one under the imap directory too. Did you get that one as well?

If you change the names and the projects, I can run through when your done and
verify that mailnews is finding the new properties files (i.e. taht we still get
the strings out correctly).

Comment 6

[tao]

Hi, Scott:


Would you verify this for me?


Thanks.

Comment 7

[Scott MacGregor]

Tao, I've verified your changes and have also started to remove the old files
from the repository. Thanks for doing this!

Comment 8

[tao]

Per mscott's comment, marking it verified.