709776 - Move tri-licensed Android library parts under mozutils

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Mike Hommey [:glandium]]

There is no real reason to keep our Gecko android c++ code under other-licenses/android when the code is tri-licensed.

Comment 1

[Mike Hommey [:glandium]]

Attachment: Move tri-licensed Android library parts under mozutils/

Comment 2

[Mike Hommey [:glandium]]

Attachment: Move tri-licensed Android library parts under mozutils/

Refreshed

Comment 3

[Mike Hommey [:glandium]]

Attachment: Move tri-licensed Android library parts under mozutils/

Edited the wrong patch on the wrong bug. Restoring the previous one.

Comment 4

[Mike Hommey [:glandium]]

Attachment: Move tri-licensed Android library parts under mozglue/.

Refreshed patch against bug 701371, which instead of just moving mozutils, also now renames it.

Comment 5

[Mike Hommey [:glandium]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/8ac6f34b44ff

Comment 6

[Mike Hommey [:glandium]]

Backed out because of random and unexplainable android bustages.
https://hg.mozilla.org/integration/mozilla-inbound/rev/db6e5b2bf92d

Comment 7

[Mike Hommey [:glandium]]

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

Comment 8

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/c899dde43144

Comment 9

[Mike Hommey [:glandium]]

Attachment: Move tri-licensed Android library parts under mozglue/ - for aurora

[Approval Request Comment]
This is a dependency for bug 683127 and bug 713228. It only moves code around.

Comment 10

[Mike Hommey [:glandium]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/3352a89d2dde