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

RESOLVED FIXED in mozilla13

Status

()

Core
Internationalization
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: glandium, Assigned: glandium)

Tracking

Trunk
mozilla13
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

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.
Assignee: nobody → mh+mozilla
Created attachment 592122 [details] [diff] [review]
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.
Attachment #592122 - Flags: review?(l10n)
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?
Attachment #592122 - Flags: review?(l10n) → review-
(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.
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.
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.
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.
Created attachment 595982 [details] [diff] [review]
Add toolkit l10n Makefiles to browser/makefiles.sh
Attachment #595982 - Flags: review?(l10n)
Attachment #592122 - Attachment is obsolete: true
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.
Summary: Building l10n for spellcheck and intl should happen from toolkit/ not $app/ → Make sure that Makefiles needed for repacks are there for xulrunner firefox repacks.
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.
Attachment #595982 - Flags: review?(l10n) → feedback+
Attachment #595982 - Flags: review?(khuey)
Attachment #595982 - Flags: review?(khuey) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/061557e27b59
https://hg.mozilla.org/mozilla-central/rev/061557e27b59
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla13
You need to log in before you can comment on or make changes to this bug.