721737 - Make sure that Makefiles needed for repacks are there for xulrunner firefox repacks.

Status: RESOLVED FIXED

(Core :: Internationalization, defect)

Description

[Mike Hommey [:glandium]]

Currently, each application puts in $app/browser/Makefile.in snippets for l10n of spellcheck and intl, but both these directories are built from toolkit. In practice, this means that building a toolkit-only locale for use with e.g. xulrunner doesn't contain these.

Comment 1

[Mike Hommey [:glandium]]

Attachment: Building l10n for spellcheck and intl should happen from toolkit/ not $app/

At the same time, don't build l10n for toolkit from browser/ when building against a libxul sdk. I didn't do the same for mobile and b2g because i doubt they build at all against a libxul sdk.

Comment 2

[Axel Hecht [:Pike]]

Comment on attachment 592122 [details] [diff] [review]
Building l10n for spellcheck and intl should happen from toolkit/ not $app/

I think this bug is WONTFIX. It's very much desired for mobile, for example, to make a call on whether spell checking by gecko is on, as it conflicts with the spellchecker on the device.

To be frank, I'm not sure why we're calling in to intl/locales, at all right now.

The LIBXUL_SDK doesn't convince me either, not sure why that would be?

Comment 3

[Mike Hommey [:glandium]]

(In reply to Axel Hecht [:Pike] from comment #2)
> Comment on attachment 592122 [details] [diff] [review]
> Building l10n for spellcheck and intl should happen from toolkit/ not $app/
> 
> I think this bug is WONTFIX. It's very much desired for mobile, for example,
> to make a call on whether spell checking by gecko is on, as it conflicts
> with the spellchecker on the device.

I don't see how that's related. A mobile build sets MOZ_SPELLCHECK in configure.in, and spellcheck l10n will be built from toolkit/locales, through the call from mobile/locales. IOW, nothing changes. Likewise for intl.

> The LIBXUL_SDK doesn't convince me either, not sure why that would be?

I build Firefox as a xulrunner application. As such, in the Firefox objdir, there is nothing under toolkit, because it's under the xulrunner objdir.

Comment 4

[Axel Hecht [:Pike]]

It's been a few years, I guess, but I recall that Benjamin wanted toolkit localization files to be part of the xulrunner-based applications and not xulrunner itself.

Comment 5

[Mike Hommey [:glandium]]

That's not possible, currently. xulrunner-based applications do not necessarily have the full source of xulrunner corresponding to the sdk they use to build, and the source is required to build l10n.

Comment 6

[Axel Hecht [:Pike]]

Talked to Mike IRL, given that the xulrunner part is hackery right now, we should just make sure that we have SUBMAKEFILES set up in browser and toolkit to not run into the build problem, and pack up all of toolkit as part of the firefox langpack.

Comment 7

[Mike Hommey [:glandium]]

Attachment: Add toolkit l10n Makefiles to browser/makefiles.sh

Comment 8

[Mike Hommey [:glandium]]

Comment on attachment 595982 [details] [diff] [review]
Add toolkit l10n Makefiles to browser/makefiles.sh

Note we may want to make that depending on LIBXUL_SDK being defined.

Comment 9

[Axel Hecht [:Pike]]

Comment on attachment 595982 [details] [diff] [review]
Add toolkit l10n Makefiles to browser/makefiles.sh

Review of attachment 595982 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me. As this is the top-level firefox build foo, you probably want the real review from a peer of build.

Comment 10

[Mike Hommey [:glandium]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/061557e27b59

Comment 11

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/061557e27b59