Closed
Bug 787241
Opened 12 years ago
Closed 12 years ago
enable android armv6 on mozilla-release (dep + releases) when fx 16 merges in
Categories
(Release Engineering :: General, defect, P3)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mozilla, Assigned: kmoir)
References
Details
(Whiteboard: [merge day])
Attachments
(4 files, 1 obsolete file)
5.38 KB,
patch
|
bhearsum
:
review+
bhearsum
:
feedback+
kmoir
:
checked-in+
|
Details | Diff | Splinter Review |
4.10 KB,
patch
|
rail
:
review+
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
1.42 KB,
patch
|
Callek
:
review-
|
Details | Diff | Splinter Review |
1.12 KB,
patch
|
bhearsum
:
review+
|
Details | Diff | Splinter Review |
This will touch mozilla/config.py, mozilla/release-fennec-mozilla-release.py, mozilla/staging_release-fennec-mozilla-release.py
Updated•12 years ago
|
Priority: -- → P3
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → kmoir
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #668168 -
Flags: feedback?
Assignee | ||
Updated•12 years ago
|
Attachment #668168 -
Flags: feedback? → feedback?(bhearsum)
Assignee | ||
Comment 2•12 years ago
|
||
I haven't done this before, not sure if I captured all the changed required.
Comment 3•12 years ago
|
||
Comment on attachment 668168 [details] [diff] [review] patch Review of attachment 668168 [details] [diff] [review]: ----------------------------------------------------------------- Yup, this looks good to me. Thanks for updating the staging config too!
Attachment #668168 -
Flags: feedback?(bhearsum) → feedback+
Comment 4•12 years ago
|
||
I'm going to ride along this fix here, because I saw the issue while I was reviewing :). We set MOZ_SYMBOLS_EXTRA_BUILDID unnecessarily for a whole bunch of platforms. Android and Android armv6 don't collide in terms of namespace on the symbol server: inflating: fennec-17.0a2-Android-20121004042009-arm-android-armv6-mozilla-aurora-symbols.txt inflating: fennec-17.0a2-Android-20121004042009-arm-mozilla-aurora-symbols.txt Further, mozilla-central is the authoritative "18.0" right now. It doesn't need anything extra set, only project branches. Same thing goes for aurora, beta, and release -- they're authoritative for their respective versions, so don't need to set anything. Also, I found a bug in the noion setup that probably caused us to lose symbol information for either armv7 or noion (depending on who uploaded first) on project branches.
Attachment #668395 -
Flags: review?(rail)
Updated•12 years ago
|
Attachment #668395 -
Flags: review?(rail)
Comment 5•12 years ago
|
||
Ted reminded me that the reason armv6 is uploading to a different place right is because this is set, let's not break that :).
Attachment #668395 -
Attachment is obsolete: true
Attachment #668404 -
Flags: review?(rail)
Updated•12 years ago
|
Attachment #668404 -
Flags: review?(rail) → review+
Updated•12 years ago
|
Attachment #668404 -
Flags: checked-in+
Updated•12 years ago
|
Attachment #668168 -
Flags: review+
Assignee | ||
Updated•12 years ago
|
Attachment #668168 -
Flags: checked-in+
Comment 6•12 years ago
|
||
in production
Assignee | ||
Comment 7•12 years ago
|
||
Attachment #668516 -
Flags: review?(bhearsum)
Assignee | ||
Comment 8•12 years ago
|
||
Attachment #668524 -
Flags: review?(bhearsum)
Updated•12 years ago
|
Attachment #668524 -
Flags: review?(bhearsum) → review+
Comment 9•12 years ago
|
||
in production
Assignee | ||
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 10•12 years ago
|
||
Comment on attachment 668516 [details] [diff] [review] patch for mozilla-tests patch was superseded (correctly in the other patch)
Attachment #668516 -
Flags: review?(bhearsum) → review-
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
Updated•6 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•