449487 - Create AMO initialization script

Status: RESOLVED FIXED

(Webtools Graveyard :: Verbatim, defect)

Description

[Wil Clouser [:clouserw]]

We need a script to setup a fresh AMO checkout for verbatim.  I'd like to be able to run this per-locale on demand from a web page.  For example, when we add a new locale it would be nice to have this run from a single click just for that locale.  In my head the script is something like:

foreach $locale in /locale/:
- run `python /scripts/convert/amo2po.py --template=$locale/LC_MESSAGES/messages.po --input=en_US/LC_MESSAGES/messages.po --output=$locale/LC_MESSAGES/verbatim.combined.po`
- rename $locale/LC_MESSAGES/messages.po to $locale/LC_MESSAGES/messages.po.orig (so verbatim doesn't try to edit it; bug 449350)
- foreach $page in /locale/$locale/pages/
    - run `python /translate/convert/html2po.py --input=$page.thtml --output=$page.po`

Comment 1

[Wil Clouser [:clouserw]]

This became a function in the amodemo.py script from bug 449488.  It's not complete because it doesn't handle /pages/ yet but it does work for the .po files.

Comment 2

[Fred Wenzel [:wenzel]]

I'll work on getting the pages/*.thtml files converted to .po and back.

Comment 3

[Wil Clouser [:clouserw]]

I think bug 449484 blocks this

Comment 4

[Fred Wenzel [:wenzel]]

Thanks. Unless we use htmltidy like I suggest in bug 449353 comment 2? I understand that would alter our pages when converting back, but -- if tidy lives up to its promises -- it would do so in a good way.

Comment 5

[Fred Wenzel [:wenzel]]

Committed as revision 12097 to the Pootle trunk. This uses HTMLTidy to do the heavy lifting.

What it does do is generate .po files from the existing pages in en_US. What it does not do is take existing, localized, pages under consideration.

For initializing a language, that is fine (which is why I am closing this bug). But for updating from SVN, this will become vital. We need to be able to allow merging existing, translated .thtml files into the .po file created from the en_US template using this script.

I'll file a new bug for this.