1395457 - De-hardcode product name when producing webext-langpack

Status: RESOLVED FIXED

(Core :: Internationalization, enhancement, P3)

Description

[Zibi Braniecki [:zbraniecki][:gandalf]]

The python code used to generate webextension langpacks' manifest.json has the name `Firefox` hardcoded in it: http://searchfox.org/mozilla-central/source/python/mozbuild/mozbuild/action/langpack_manifest.py

We should de-hardcode it, but I'm not sure where can we take the brandName from.

Comment 1

[Zibi Braniecki [:zbraniecki][:gandalf]]

We currently have one place where we use the name "Firefox" to build the description of the language pack "Language pack for Firefox for ab-CD"[0]

I'd like to de-hardcode the name, so that we can create a language pack for Firefox or for Fennec or for Thunderbird, Seamonkey etc.

The challenge here is that I don't know where to take the product name from in mozbuild/action.

I could maybe use .properties parser and look for brand.properties? Or try to vendor fluent python parser and add brand.ftl already.

Alternatively, I can remove the product specifier and keep it as "Language pack for ab-CD".

Pike, what do you think?

[0] http://searchfox.org/mozilla-central/source/python/mozbuild/mozbuild/action/langpack_manifest.py#204

p.s. additionally, if I had the product name, I could increase the specificity of the langpack addon ID, by replacing "[e-mail]" with "[e-mail]" and "[e-mail]" or sth like that.

Comment 2

[Axel Hecht [:Pike]]

MOZ_APP_DISPLAYNAME or MOZ_APP_NAME ? Try with devedition branding, those are the trickiest.

Probably not a good idea to use those for the langpack ID, though.

Comment 3

[Zibi Braniecki [:zbraniecki][:gandalf]]

Attachment: Bug 1395457 - De-hardcode product name when producing webext-langpack.

Review commit: https://reviewboard.mozilla.org/r/175138/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/175138/

Comment 4

[Zibi Braniecki [:zbraniecki][:gandalf]]

Awesome. I used MOZ_APP_DISPLAYNAME.

I'm not going to touch the langpack ID for now.

Comment 5

[Axel Hecht [:Pike]]

Comment on attachment 8903337 [details]
Bug 1395457 - De-hardcode product name when producing webext-langpack.

https://reviewboard.mozilla.org/r/175138/#review181020

I think you'll need a different reviewer to guide you to the right thing here.

MOZ_APP_DISPLAYNAME doesn't work out of the box. Thunderbird uses Daily, for example, and I'm not convinced that our langpack titles should differ for Developer Edition, or Nightly for that matter?

Maybe there's nothing good yet, and we need to explicitly create something for our use-case.

Comment 6

[Zibi Braniecki [:zbraniecki][:gandalf]]

Kris, Dave, any thoughts on this?

I'm trying to pick a good description for the new langpacks. Using MOZ_APP_DISPLAYNAME gives me: "Language pack for Firefox for pl" but as Pike pointed out also "Language pack for Nightly for pl" and "Language pack for Daily for pl".

I'd be happy to not differentiate between channels, but MOZ_APP_NAME is all lower-case, so not that good for "Language pack for firefox for pl".

Any idea what should I do, and if Axel is correct that we need to generate a new one, where should I take it from?

Comment 7

[Kris Maglione [:kmag]]

MOZ_APP_DISPLAYNAME (or the equivalent brandShortName from a string bundle) is probably the best we're going to do. We generally try to consistently use the appropriate codename for unbranded/unofficial/nightly builds, so there aren't any easy ways to access the branded app name in those. And we probably want to stick to the code name in the langpack name for consistency in those builds, anyway.

Comment 8

[Dave Townsend [:mossop]]

I don't think I have anything else to add here.

Comment 9

[Zibi Braniecki [:zbraniecki][:gandalf]]

Comment on attachment 8903337 [details]
Bug 1395457 - De-hardcode product name when producing webext-langpack.

Ok, Pike is ok with Kris' approach. Setting r?kmag

Comment 10

[Kris Maglione [:kmag]]

Comment on attachment 8903337 [details]
Bug 1395457 - De-hardcode product name when producing webext-langpack.

https://reviewboard.mozilla.org/r/175138/#review182460

Comment 11

[Mozilla Autoland]

We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s 57d24857e849 -d 95543c372e9f: rebasing 418651:57d24857e849 "Bug 1395457 - De-hardcode product name when producing webext-langpack. r=kmag" (tip)
merging python/mozbuild/mozbuild/action/langpack_manifest.py
merging toolkit/locales/l10n.mk
warning: conflicts while merging python/mozbuild/mozbuild/action/langpack_manifest.py! (edit, then use 'hg resolve --mark')
warning: conflicts while merging toolkit/locales/l10n.mk! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)

Comment 12

[Zibi Braniecki [:zbraniecki][:gandalf]]

Comment on attachment 8903337 [details]
Bug 1395457 - De-hardcode product name when producing webext-langpack.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/175138/diff/1-2/

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3a39f8baf4ec
De-hardcode product name when producing webext-langpack. r=kmag

Comment 14

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/3a39f8baf4ec