Need a script which uses the Rosetta internal XML parser to check upstream Mozilla translations

Bug #503382 reported by Arne Goetje
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Wishlist
Jeroen T. Vermeulen

Bug Description

Rationale:
Currently when uploading Mozilla translations into Rosetta, certain languages fail to import, because of syntax or format bugs in the .dtd or .manifest files included in the XPI tarball. These bugs do not get detected by upstream developers/translators, since Firefox is more tolerant, while Rosetta's XPI parser bails out.
I currently fix those issues manually, rebuild the XPI tarballs and upload them again to Rosetta until they get accepted. This sometimes needs multiple attempts, since there are often multiple bugs in the XPI tarballs and the XML parser bails out at the first one it finds.
What's more, those bugs, even when sent with patches to upstream, don't get fixed in the actual stable release which we are importing in Rosetta, but with chance in the current development tree. However, since the directory structure in the XPI tarballs and the upstream Mercurial VCS don't match, the patches I generate against the XPI tarballs are pretty useless for upstream translators.
Therefor I'd like to be able to check all .dtd and .manifest files in the upstream VCS with the Launchpad XML parser to be able to fix those issues before the translations get released in the next stable release.

Since Launchpad wants to import upstream sources, I think this script would be a very useful asset, too.

Related branches

Changed in rosetta:
importance: Undecided → Medium
assignee: nobody → Jeroen T. Vermeulen (jtv)
status: New → In Progress
milestone: none → 10.01
Changed in rosetta:
importance: Medium → Wishlist
summary: - [Wishlist] Need a script which uses the Rosetta internal XML parser to
- check upstream Mozilla translations
+ Need a script which uses the Rosetta internal XML parser to check
+ upstream Mozilla translations
Revision history for this message
Ursula Junque (ursinha) wrote : A commit mentioned this bug
Changed in rosetta:
status: In Progress → Fix Committed
tags: added: qa-needstesting
Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

Note: gave this wishlist priority because it was used in the original bug title, but implemented it anyway because it was particularly valuable to integration between Launchpad/Ubuntu and upstream.

So quick in-focus payoff for little cost.

Revision history for this message
Ursula Junque (ursinha) wrote :
Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

Arne seems to like the new script. Marking this as QA'ed.

tags: added: qa-ok
removed: qa-needstesting
Curtis Hovey (sinzui)
Changed in rosetta:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.