Closed
Bug 709776
Opened 13 years ago
Closed 13 years ago
Move tri-licensed Android library parts under mozutils
Categories
(Firefox Build System :: General, defect)
Tracking
(firefox11 fixed)
RESOLVED
FIXED
mozilla12
Tracking | Status | |
---|---|---|
firefox11 | --- | fixed |
People
(Reporter: glandium, Assigned: glandium)
References
Details
(Whiteboard: [inbound])
Attachments
(2 files, 3 obsolete files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
glandium
:
review+
blassey
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
There is no real reason to keep our Gecko android c++ code under other-licenses/android when the code is tri-licensed.
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → mh+mozilla
Assignee | ||
Updated•13 years ago
|
Attachment #580903 -
Attachment is obsolete: true
Attachment #580903 -
Flags: review?(blassey.bugs)
Assignee | ||
Comment 3•13 years ago
|
||
Edited the wrong patch on the wrong bug. Restoring the previous one.
Attachment #580991 -
Flags: review?(blassey.bugs)
Assignee | ||
Updated•13 years ago
|
Attachment #580982 -
Attachment is obsolete: true
Updated•13 years ago
|
Attachment #580991 -
Flags: review?(blassey.bugs) → review+
Assignee | ||
Comment 4•13 years ago
|
||
Refreshed patch against bug 701371, which instead of just moving mozutils, also now renames it.
Assignee | ||
Updated•13 years ago
|
Attachment #580991 -
Attachment is obsolete: true
Assignee | ||
Comment 5•13 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/8ac6f34b44ff
Whiteboard: [inbound]
Assignee | ||
Comment 6•13 years ago
|
||
Backed out because of random and unexplainable android bustages. https://hg.mozilla.org/integration/mozilla-inbound/rev/db6e5b2bf92d
Whiteboard: [inbound]
Assignee | ||
Comment 7•13 years ago
|
||
The Android bustages were due to the missing ANDROID_ARM_LINKER definition in mozglue/android/Makefile.in (it's the only DEFINE from other-licenses/android/Makefile.in that is used in APKOpen.cpp). The result is that we were missing the cache flushes from bug 690856 and were hitting it again. https://hg.mozilla.org/integration/mozilla-inbound/rev/c899dde43144
Assignee | ||
Updated•13 years ago
|
Whiteboard: [inbound]
Comment 8•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/c899dde43144
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
Assignee | ||
Comment 9•13 years ago
|
||
[Approval Request Comment] This is a dependency for bug 683127 and bug 713228. It only moves code around.
Attachment #591485 -
Flags: review+
Attachment #591485 -
Flags: approval-mozilla-aurora?
Updated•13 years ago
|
Attachment #591485 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 10•13 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/3352a89d2dde
status-firefox11:
--- → fixed
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•