387437 - specify wanted app to l10n.py

Status: RESOLVED FIXED

(Core :: Internationalization: Localization, defect)

Description

[Robert Kaiser]

<KaiRo> Pike: hmm, l10n.py only does a Firefox L10n, but I'd need a SeaMonkey one
<Pike> KaiRo: that's a one-line change (and a missed commandline option, file a bug?)

Comment 1

[Axel Hecht [:Pike]]

Attachment: add --app commandline support

just add a commandline option for --app

Now you can do 

python l10n.py --app=browser --app=mail foo-BAR

Comment 2

[Robert Kaiser]

Comment on attachment 271659 [details] [diff] [review]
add --app commandline support

Most of this looks fine to me (though I don't really understand python) and it works for even cases with multiple app, but it fails for the default case of not defining --app at all:
------------------
robert@robert:/mnt/mozilla/src/mozilla> python tools/l10n/l10n.py bla
Traceback (most recent call last):
  File "tools/l10n/l10n.py", line 153, in <module>
    opts.app, exceptions=exceptions)
  File "tools/l10n/l10n.py", line 79, in createLocalization
    if not len(apps):
TypeError: object of type 'NoneType' has no len()
------------------

I guess you need a different way of checking for this being not specified...

Comment 3

[Axel Hecht [:Pike]]

Attachment: catch default for real

Duh. Fixed.

Comment 4

[Robert Kaiser]

Comment on attachment 271665 [details] [diff] [review]
catch default for real

Yes, this seems to work fine now. r=me

Comment 5

[Axel Hecht [:Pike]]

Thanks for catching the bug, and the review. FIXED.

Checking in l10n.py;
/cvsroot/mozilla/tools/l10n/l10n.py,v  <--  l10n.py
new revision: 1.2; previous revision: 1.1
done