260806 - Allow localization of README.txt

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[Pavel Franc - Mozilla.cz]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; rv:1.7.3) Gecko/20040913 Firefox/0.10
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; rv:1.7.3) Gecko/20040913 Firefox/0.10

To allow localization of README.txt move it into l10n tree

http://lxr.mozilla.org/aviarybranch/source/browser/app/README.txt

Reproducible: Always
Steps to Reproduce:
1.
2.
3.

Comment 1

[chris hofmann]

chase/granrose can you help on this one?

Comment 2

[Chase Phillips]

choffman: Sure thing.

Pawell: Where do you recommend this file go in the mozilla source tree?  Also,
what builds is this file a part of?  We're not certain how/where it's being
packaged right now.

Comment 3

[Pavel Franc - Mozilla.cz]

This file is part of firefox browser.xpi (bin/README.txt) package which should
be locale independent. The idea is to move the file from 
/mozilla/browser/app/ into 
/mozilla/browser/locales/en-US/ and every /l10n/browser/locales/@AB-CD@/ 
and then packed it into @AB-CD@.jar (bin\README.txt)

Thus you have to remove corresponding lines form
/mozilla/browser/app/Makefile.in and add them into
/mozilla/browser/locales/Makefile.in  

This is just my idea as a l10n guy, it must be approved/corrected/dumped by
Benjamin.

Comment 4

[Pavel Franc - Mozilla.cz]

Related bug 263446

Comment 5

[chris hofmann]

wonder if we could put a copy of the file in mozilla/browser/locales/en-US/ as
step one to allow translation work to get going, and then follow up with mods to
the build and installer packaging systems to start to pick up the file from the
new location... 

Comment 6

[:deanis74]

This was fixed on the branch -- did it get reviews and approval?

Comment 7

[Vincent Béron]

Which character encoding to use for README.txt?

Comment 8

[Chase Phillips]

Dean: The fix was discussed and reviewed by bsmedberg last week.  I just got
around to implementing it today.

I meant to post my write-up of the fix sooner.  Here it is:

Moved README.txt from browser/app/ to browser/locales/en-US/.  Updated
browser/app/Makefile.in to remove install of README.txt during libs target. 
Updated browser/locales/Makefile.in to handle installing README.txt for every
locale instance.