Closed
Bug 775232
Opened 12 years ago
Closed 12 years ago
Set up build configs in order to produce ARMv6 builds on mozilla-beta (Firefox 16)
Categories
(Release Engineering :: Release Automation: Other, defect)
Tracking
(firefox16 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox16 | --- | fixed |
People
(Reporter: blassey, Assigned: mozilla)
References
Details
Attachments
(5 files, 3 obsolete files)
1.08 KB,
patch
|
mozilla
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
2.32 KB,
patch
|
armenzg
:
review+
akeybl
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
2.25 KB,
patch
|
armenzg
:
review+
akeybl
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
6.58 KB,
patch
|
Details | Diff | Splinter Review | |
5.32 KB,
patch
|
mozilla
:
review+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
No description provided.
Comment 1•12 years ago
|
||
Armen, you set these up originally, did we forget to do something when the merged happened on Monday?
Component: Release Engineering: Automation (Release Automation) → Release Engineering: Automation (General)
QA Contact: bhearsum → catlee
Comment 2•12 years ago
|
||
They got enabled up to FF16 (m-a). We can enabled them on m-b if blassey thinks that we're good. Enabling them on mozilla-beta does *not* mean we will be doing betas for it... yet.
Comment 3•12 years ago
|
||
We can enable builds on check-in but asking for armv6 builds to be part of the "betas" is another story that needs someone to be pulled out to do a staging release.
Attachment #643567 -
Flags: review?(aki)
Assignee | ||
Comment 4•12 years ago
|
||
Comment on attachment 643567 [details] [diff] [review] add armv6 builds on checkin for mozilla-beta r=me if you s,FF17,FF15,
Attachment #643567 -
Flags: review?(aki) → review+
Comment 5•12 years ago
|
||
Comment on attachment 643567 [details] [diff] [review] add armv6 builds on checkin for mozilla-beta This should go in tomorrow's reconfig: http://hg.mozilla.org/build/buildbot-configs/rev/1d9ccb639b45
Attachment #643567 -
Flags: checked-in+
Comment 6•12 years ago
|
||
In production
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → aki
Component: Release Engineering: Automation (General) → Release Engineering: Automation (Release Automation)
QA Contact: catlee → bhearsum
Assignee | ||
Comment 7•12 years ago
|
||
These builds are currently en-US only. I'll proceed with en-US for the first beta. Multilocale and/or single locale repacks will take further configuration, and we should turn that on in nightlies first.
Assignee | ||
Comment 8•12 years ago
|
||
With the branding as-is, this will have to be published into the same app (org.mozilla.firefox_beta). That means our manifest has to be set correctly, and we will probably require the same ugly versionCode increment dance that we currently have for android-xul.
Assignee | ||
Comment 9•12 years ago
|
||
Without this, automated tests we run will be perma-broken.
Attachment #644470 -
Flags: review?(armenzg)
Assignee | ||
Comment 10•12 years ago
|
||
Attachment #644477 -
Flags: review?(armenzg)
Assignee | ||
Comment 11•12 years ago
|
||
Currently sign_android.py assumes that all platforms will have the same apk_base_name. The apk name of android-armv6 apks is different (fennec-version.locale.android-arm-armv6.apk). We'll either need 2 sets of config files + 2 passes of signing, or we need to add per-platform apk_base_name support (and per-platform locale support, while we're at it).
Assignee | ||
Comment 12•12 years ago
|
||
Passes checkconfig; haven't seen it build.
Assignee | ||
Comment 13•12 years ago
|
||
Attachment #644537 -
Attachment is obsolete: true
Assignee | ||
Comment 14•12 years ago
|
||
Added android-armv6 branding changes to https://wiki.mozilla.org/Release_Management/Merge_Documentation for both aurora and beta.
Assignee | ||
Comment 15•12 years ago
|
||
(In reply to Aki Sasaki [:aki] from comment #8) > With the branding as-is, this will have to be published into the same app > (org.mozilla.firefox_beta). That means our manifest has to be set > correctly, and we will probably require the same ugly versionCode increment > dance that we currently have for android-xul. From Blassey: "The package name being the same is exactly what we need. The only over requirement is that the ARMv6 build id be at least one less than the ARMv7 build." We have a hack in embedding/android/Makefile.in to automatically increment the ANDROID_VERSION_CODE. I can't currently find a corresponding hack to automatically decrement the ANDROID_VERSION_CODE for armv6. This means we can't actually build armv6 at the same time as armv7; we have to build it in the previous hour. This will be a pretty large sticking point. While we're in here mucking with this stuff, this should be based on the buildid that we pass in instead of just basing off of the machine time, otherwise we're going to hit issues where a delay makes the versionCode overlap the wrong way. The above blocks our ability to ship armv6 beta builds.
Assignee | ||
Comment 16•12 years ago
|
||
This is the quick n ugly fix, where I have a 2nd set of config files and we have to sign twice. The other fix would be having per-platform locales and apk names, but would be a much larger patch.
Comment 17•12 years ago
|
||
FTR the packaging patches that landed on mozilla-aurora have not been ported to mozilla-beta. Candidates for this porting are bug 757909, bug 761454, bug 766664, bug 767864 and bug 768386. blassey ^
Updated•12 years ago
|
Attachment #644470 -
Flags: review?(armenzg) → review+
Updated•12 years ago
|
Attachment #644477 -
Flags: review?(armenzg) → review+
Updated•12 years ago
|
Whiteboard: work blocked on findings by blassey on comment 15
Reporter | ||
Comment 18•12 years ago
|
||
requested approval to land all 4 of those bugs (5 patches total) on beta
Assignee | ||
Comment 19•12 years ago
|
||
Comment on attachment 644470 [details] [diff] [review] armv6 beta branding on m-b [Approval Request Comment] Bug caused by (feature/regressing bug #): Turning on armv6 builds where there were none before. User impact if declined: Automated tests will all be broken. Testing completed (on m-c, etc.): None. Risk to taking this patch (and alternatives if risky): Low. String or UUID changes made by this patch: None.
Attachment #644470 -
Flags: approval-mozilla-beta?
Assignee | ||
Comment 20•12 years ago
|
||
Comment on attachment 644477 [details] [diff] [review] aurora branding for m-a [Approval Request Comment] Bug caused by (feature/regressing bug #): Turning on armv6 builds on aurora, not noticing they were nightly-branded. User impact if declined: Automated tests will all be broken. Testing completed (on m-c, etc.): None. Risk to taking this patch (and alternatives if risky): Low. String or UUID changes made by this patch: None.
Attachment #644477 -
Flags: approval-mozilla-aurora?
Updated•12 years ago
|
Attachment #644477 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 21•12 years ago
|
||
Comment on attachment 644470 [details] [diff] [review] armv6 beta branding on m-b [Triage Comment] Approving for Beta as well, assuming that this is NPOTB (since we're not yet building ARMv6 on Beta).
Attachment #644470 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Assignee | ||
Comment 22•12 years ago
|
||
(In reply to Alex Keybl [:akeybl] from comment #21) > Comment on attachment 644470 [details] [diff] [review] > armv6 beta branding on m-b > > [Triage Comment] > Approving for Beta as well, assuming that this is NPOTB (since we're not yet > building ARMv6 on Beta). We are actually building armv6 on beta, once https://bugzilla.mozilla.org/attachment.cgi?id=643567 landed and buildbot reconfiged. We turned it on since there's a push to build armv6 in Fx15.0b2. It's perma-red, however, due to comment 17. https://tbpl.mozilla.org/?tree=Mozilla-Beta&noignore=1
Assignee | ||
Comment 23•12 years ago
|
||
Comment on attachment 644547 [details] [diff] [review] (needs testing) [mozharness] android-armv6 signing Hoping to use the patch in https://bugzilla.mozilla.org/show_bug.cgi?id=750976 instead.
Attachment #644547 -
Attachment is obsolete: true
Comment 24•12 years ago
|
||
Moved to Firefox 16.
Summary: Set up build configs in order to produce ARMv6 builds on mozilla-beta (Firefox 15) → Set up build configs in order to produce ARMv6 builds on mozilla-beta (Firefox 16)
Comment 25•12 years ago
|
||
Refreshed patch tested in staging
Attachment #644539 -
Attachment is obsolete: true
Blocks: mobile-automation
Comment 26•12 years ago
|
||
Hi guys any news on when these fix reach firefox beta on the google play store ?
Comment 27•12 years ago
|
||
(In reply to lord.dhruv from comment #26) > Hi guys any news on when these fix reach firefox beta on the google play > store ? This is probably a question better asked on the dev.apps.firefox newsgroup (https://groups.google.com/forum/?fromgroups#!forum/mozilla.dev.apps.firefox), but Firefox for Armv6 is scheduled to ship with 16, which will be released on October 9th.
Assignee | ||
Comment 28•12 years ago
|
||
Attachment #655747 -
Flags: review+
Assignee | ||
Comment 29•12 years ago
|
||
Comment on attachment 655747 [details] [diff] [review] rail's configs, unbitrotted http://hg.mozilla.org/build/buildbot-configs/rev/5f955db73ffa
Attachment #655747 -
Flags: checked-in+
Assignee | ||
Updated•12 years ago
|
Whiteboard: work blocked on findings by blassey on comment 15
Assignee | ||
Comment 30•12 years ago
|
||
Branding appears correct. Resolving.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
status-firefox16:
--- → fixed
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•