Closed Bug 709776 Opened 12 years ago Closed 12 years ago

Move tri-licensed Android library parts under mozutils

Categories

(Firefox Build System :: General, defect)

ARM
Android
defect
Not set
normal

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)

There is no real reason to keep our Gecko android c++ code under other-licenses/android when the code is tri-licensed.
Attachment #580903 - Flags: review?(blassey.bugs)
Assignee: nobody → mh+mozilla
Blocks: 683127
Attachment #580903 - Attachment is obsolete: true
Attachment #580903 - Flags: review?(blassey.bugs)
Edited the wrong patch on the wrong bug. Restoring the previous one.
Attachment #580991 - Flags: review?(blassey.bugs)
Attachment #580982 - Attachment is obsolete: true
Attachment #580991 - Flags: review?(blassey.bugs) → review+
Refreshed patch against bug 701371, which instead of just moving mozutils, also now renames it.
Attachment #580991 - Attachment is obsolete: true
Backed out because of random and unexplainable android bustages.
https://hg.mozilla.org/integration/mozilla-inbound/rev/db6e5b2bf92d
Whiteboard: [inbound]
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
Whiteboard: [inbound]
https://hg.mozilla.org/mozilla-central/rev/c899dde43144
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
[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?
Attachment #591485 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.