1123428 - Let developers localize add-on metadata with jpm

Status: RESOLVED WORKSFORME

(Add-on SDK Graveyard :: General, enhancement, P1)

Description

[Adrian Aichner [:anaran]]

+++ This bug was initially created as a clone of Bug #661083 +++

While name, description localization gets picked up into .xpi package when building with
cfx xpi
I don'see this happening for
jpm xpi

I use this to test:

locale/en-US.properties:

[e-mail] = title-en
[e-mail] = author-en
[e-mail] = description-en
[e-mail] = homepage-en

Comment 1

[Erik Vold [:erikvold] (please needinfo? me)]

Hmm it looks like I made a mistake with https://github.com/mozilla/addon-sdk/tree/master/test/addons/l10n-manifest when landing bug 661083, this test add-on should have required a preferences file as is stated in https://developer.mozilla.org/en-US/docs/Mozilla/Localization/Localizing_extension_descriptions#Localizable_Lists

I landed bug 661083 in haste because I thought we could support that use case and I knew that we were not going to release another version of cfx, which we still do not plan to do, so I figured I could land the test and implement support for native jetpacks (which are still wip) later.

I've thought about this implementation more and I realize it's much more work than I thought to support the method described here https://developer.mozilla.org/en-US/docs/Mozilla/Localization/Localizing_extension_descriptions#Localizable_Lists with the sdk localization file structure described here https://developer.mozilla.org/en-US/Add-ons/SDK/Tutorials/l10n

We can support https://developer.mozilla.org/en-US/Add-ons/SDK/Tutorials/l10n with jpm without issue, which I have tests for https://github.com/mozilla/jpm/tree/master/test/addons/l10n-properties already.

So to summarize, we can support the l10n features in the last release of cfx, which did not include the manifest localization, and supporting this new manifest localization feature introduced in bug 661083 may delay native jetpacks.  So I think we should revert 661083, and reconsider here if that design is still one that we want to pursue, this way we can not further delay native jetpacks.

Making this bug depend on native-jetpacks is a good call.

Comment 2

[Adrian Aichner [:anaran]]

Let me summarize further to verify my understanding:

Instead of supporting this cfx feature in jpm as well you want to revert this cfx feature instead?

Leaving us with no such feature?

I hope I am wrong.

Comment 3

[Adrian Aichner [:anaran]]

https://github.com/mozilla/addon-sdk/tree/master/test/addons/l10n-manifest
is a 404 for me ATM.

Comment 4

[Erik Vold [:erikvold] (please needinfo? me)]

(In reply to adrian.aichner from comment #2)
> Let me summarize further to verify my understanding:
> 
> Instead of supporting this cfx feature in jpm as well you want to revert
> this cfx feature instead?

The feature never existed in cfx, and we never planned for that to be the case, since we do not plan to release a new version of cfx.

This is the last release of cfx https://github.com/mozilla/addon-sdk/tree/1.17, you can see that the feature does no exist there either: https://github.com/mozilla/addon-sdk/tree/1.17/test/addons/l10n-manifest

I'm sorry this means manifest localization will not be supported as soon as we had hoped, but the good news is that jpm and native jetpack will be released and available that much sooner.

Comment 5

[Erik Vold [:erikvold] (please needinfo? me)]

I made a bug about this issue for npm, and I think we should just implement whatever they end up doing, see https://github.com/npm/newww/issues/562

Comment 6

[Andy McKay]

This bug might be fixed on jpm and all the issues are tracked on github, please see:

https://github.com/mozilla-jetpack/jpm/issues/479

We think this is fixed and needs documentation on MDN.

Comment 7

[Sebastian Zartner [:sebo]]

This one and bug 1218719 sound very similar. Should we merge them?

It looks like the implementation differs from the originally requested one. Asking for clarification:

https://github.com/mozilla-jetpack/jpm/pull/495#issuecomment-215031708

Sebastian