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

RESOLVED FIXED in mozilla13

Status

()

Core
Internationalization
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: glandium, Assigned: glandium)

Tracking

Trunk
mozilla13
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

6 years ago
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)

Updated

6 years ago
Assignee: nobody → mh+mozilla
(Assignee)

Comment 1

6 years ago
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 2

6 years ago
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-
(Assignee)

Comment 3

6 years ago
(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

6 years ago
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.
(Assignee)

Comment 5

6 years ago
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

6 years ago
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.
(Assignee)

Comment 7

6 years ago
Created attachment 595982 [details] [diff] [review]
Add toolkit l10n Makefiles to browser/makefiles.sh
Attachment #595982 - Flags: review?(l10n)
(Assignee)

Updated

6 years ago
Attachment #592122 - Attachment is obsolete: true
(Assignee)

Comment 8

6 years ago
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.

Updated

6 years ago
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 9

6 years ago
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+
(Assignee)

Updated

6 years ago
Attachment #595982 - Flags: review?(khuey)
Attachment #595982 - Flags: review?(khuey) → review+
(Assignee)

Comment 10

6 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/061557e27b59
https://hg.mozilla.org/mozilla-central/rev/061557e27b59
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla13
You need to log in before you can comment on or make changes to this bug.