Closed Bug 707577 Opened 13 years ago Closed 12 years ago

Create a |fast-package| target on mobile

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla15

People

(Reporter: mounir, Assigned: mounir)

References

Details

Attachments

(2 files)

Attached patch PatchSplinter Review
This target will be used (maybe temporarily) to trigger speed optimizations when packaging.
Attachment #578946 - Flags: review?(khuey)
Blocks: 707569
Comment on attachment 578946 [details] [diff] [review]
Patch

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

I don't think we should have a separate target.  Either we're convinced enough of a change to do it for regular packaging or we shouldn't be imposing it on developers.
Attachment #578946 - Flags: review?(khuey) → review-
Comment on attachment 578946 [details] [diff] [review]
Patch

Ok, after looking at the optimizations you have in mind I changed my mind.
Attachment #578946 - Flags: review- → review+
http://hg.mozilla.org/integration/mozilla-inbound/rev/5667275e8112

Should the same change be made to mobile/android/build.mk ?
Yeah, that seems more useful nowadays...
https://hg.mozilla.org/mozilla-central/rev/5667275e8112
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
(In reply to Nick Thomas [:nthomas] from comment #3)
> Should the same change be made to mobile/android/build.mk ?

Indeed.
Over to you to do that.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Adds the same target for the native android app. Also fiddles the previously added target to pass the variable as a makefile var, not an environment var.
Attachment #620254 - Flags: review?(khuey)
Target Milestone: --- → mozilla15
https://hg.mozilla.org/mozilla-central/rev/baa69540739a
Status: REOPENED → RESOLVED
Closed: 12 years ago12 years ago
Resolution: --- → FIXED
Depends on: 835150
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: