824169 - Remove old Android NDKs

Status: RESOLVED WONTFIX

(Infrastructure & Operations Graveyard :: CIDuty, task)

Description

[Rail Aliiev [:rail]]

In bug 769099 we installing a new android ndk side by side with older ones. It'll be great to remove the old one from buildbot-configs when the new one hits all branches.

Comment 1

[Kartikaya Gupta (email:[e-mail])]

For the record, as of FF20, all of the fennec builds are using NDK r8c. Once 20 is on release we should be able to remove the r5c and r7b NDKs that were previously being used. (I assume they're not used for b2g builds).

Comment 2

[Kartikaya Gupta (email:[e-mail])]

Note that bug 832942 appears to be a problem introduced by the NDK switchover, and we may have to revert the ARMv6 build back to r5c so until that is sorted out we shouldn't get rid of the old NDKs.

Comment 3

[Rail Aliiev [:rail]]

kats, I'm not sure if bug 851291 is a real stopper here. Should I proceed and remove the old SDK after next merge?

Comment 4

[Kartikaya Gupta (email:[e-mail])]

Yeah, I don't think bug 851291 should block this. I think removing the old NDK (note: NDK, not SDK) after the next merge should be fine. I've removed the dependency with a comment in bug 851291.

Comment 5

[Rail Aliiev [:rail]]

Attachment: remove ndk5 and ndk7

To be landed after m-b merges to m-r.

Simple test:
for b in mozilla-{aurora,beta}; do
  for a in android android-armv6 android-noion android-x86; do
    for m in debug l10n-nightly l10n-release nightly release; do
       url="http://hg.mozilla.org/releases/$b/raw-file/default/mobile/android/config/mozconfigs/$a/$m"
       wget -qO - $url| grep android-ndk
    done
  done
done | sort -u

ac_add_options --with-android-ndk="/tools/android-ndk-r8c"

Comment 6

[Rail Aliiev [:rail]]

Comment on attachment 729496 [details] [diff] [review]
remove ndk5 and ndk7

Thanks to Callek, he reminded me that we use the old NDks on the b2g18 branch.

Comment 7

[Chris Cooper [:coop] (he/him)]

(In reply to Rail Aliiev [:rail] from comment #6)
> Comment on attachment 729496 [details] [diff] [review]
> remove ndk5 and ndk7
> 
> Thanks to Callek, he reminded me that we use the old NDks on the b2g18
> branch.

So is this WONTFIX then, or merely in a holding pattern until the EOL for b2g18?