1219581 - Fennec 42 RC uses wrong channel and branding

Status: VERIFIED FIXED

(Firefox Build System :: Android Studio and Gradle Integration, defect)

Description

[Ioana Chiorean]

Steps to reproduce:
1. Install Fennec 42 RC build: http://ftp.mozilla.org/pub/mobile/candidates/42.0-candidates/build2

Expected results:
- The Fennec 42 RC build should have release branding and to use release channel.

Actual results:
- The Fennec 42 RC has the Beta branding and it uses the Beta channel too ( it installs over Beta build)

Comment 1

[Sylvestre Ledru [:Sylvestre]]

ok, thanks. We will address this issue when Rail wakes up!

Comment 2

[Rail Aliiev [:rail]]

tl;dr: we picked a wrong changeset to build with and need to go with build3. Instead of 8e6007ac23a1 we should have used 7c0672cba1de.

Long version:

https://treeherder.mozilla.org/#/jobs?repo=mozilla-release&revision=7c0672cba1de contains 2 changesets, one with actual fix and the next one with a merge of heads. Without that merge we get beta mozconfigs:

$ hg cat -r c2e33a0e87d6 mobile/android/config/mozconfigs/android-api-11/release | grep branding
ac_add_options --with-branding=mobile/android/branding/beta
ac_add_options --enable-official-branding

vs

$ hg cat -r  7c0672cba1de  mobile/android/config/mozconfigs/android-api-11/release | grep branding
ac_add_options --with-branding=mobile/android/branding/official
ac_add_options --enable-official-branding

Comment 3

[Sylvestre Ledru [:Sylvestre]]

Build retriggered ~1 hour ago. Should be fine. Thanks rail!

Comment 4

[Ioana Chiorean]

Verified Fixed with http://ftp.mozilla.org/pub/mobile/candidates/42.0-candidates/build3

Comment 5

[Rail Aliiev [:rail]]

Woot! Thanks a lot for catching this edge case.

We can probably add a release sanity check to guard us against this kind of mistakes. Let me file a bug. :)