[Dolphin] Create Dolphin builds for 1.4

RESOLVED FIXED

Status

Release Engineering
General Automation
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jsmith, Assigned: catlee)

Tracking

unspecified
Dependency tree / graph

Firefox Tracking Flags

(b2g-v1.4 fixed)

Details

(Whiteboard: [Dolphin_1.4])

Attachments

(4 attachments, 1 obsolete attachment)

(Reporter)

Description

3 years ago
We need to start generating builds for Dolphin on 1.4.
(Reporter)

Updated

3 years ago
Depends on: 1011131

Updated

3 years ago
Summary: Create Dolphin builds for 1.4 → [Dolphin] Create Dolphin builds for 1.4
Whiteboard: [Dolphin_1.4]

Comment 1

3 years ago
There's no special rules to build Dolphin this time.
Here's the steps I used.

BRANCH=v1.4 ./config.sh dolphin
./build.sh kernelheader && ./build.sh

There's no need to assign ANDROIDFS_DIR for Dolphin devices.

Updated

3 years ago
Assignee: nobody → sbruno
(Assignee)

Updated

3 years ago
Depends on: 1014102
(Assignee)

Comment 2

3 years ago
Created attachment 8426412 [details]
sources.xml

initial manifest for dolphin. needs landing on mozilla-b2g30_v1_4 as b2g/config/dolphin/sources.xml
(Assignee)

Comment 3

3 years ago
Created attachment 8426527 [details] [diff] [review]
b2g bumper configs for dolphin; make it easier to get missing repos

I added some support in b2g_bumper.py to make it easier to get new devices up and running.

e.g.

 b2g_bumper.py -c b2g_bumper/1.4.py --massage-manifests --checkout-manifests --no-write --device dolphin

gave me the output needed to file bug 1014102 for missing git mirrors.
Attachment #8426527 - Flags: review?(aki)

Comment 4

3 years ago
Comment on attachment 8426527 [details] [diff] [review]
b2g bumper configs for dolphin; make it easier to get missing repos

I use self.add_failure() for this type of thing (detecting failure for later, but not fatal()ing yet), but this works too.

> +                self.info("needs sync? %s/%s -> %s/%s" % (orig_fetch_url, name, new_fetch_url, name))

Is self.info() the right level? I'd guess warning/error/critical, but since there's a fatal() afterwards, it's a matter of taste.
Attachment #8426527 - Flags: review?(aki) → review+
(Assignee)

Comment 5

3 years ago
Created attachment 8426532 [details] [diff] [review]
sources.xml in patch form
Attachment #8426412 - Attachment is obsolete: true
https://hg.mozilla.org/releases/mozilla-b2g30_v1_4/rev/123395cf1e6c
status-b2g-v1.4: --- → fixed
(Assignee)

Updated

3 years ago
Attachment #8426527 - Flags: checked-in+
Something here went live today
Catlee provided to me some details related to this (and more generally to b2g builds - thanks catlee!), I think I now have all the info I need. I'll upload patches as soon as I have some green runs in my test env.
(Assignee)

Comment 9

3 years ago
We're seeing a few build errors in staging:

12:55:13    ERROR -  hardware/libhardware_legacy/audio/AudioPolicyManagerBase.cpp:246:27: error: 'FFOS_AUDIO_DEVICE_OUT_FM' was not declared in this scope
12:55:13     INFO -  hardware/libhardware_legacy/audio/AudioPolicyManagerBase.cpp: In member function 'virtual void android_audio_legacy::AudioPolicyManagerBase::setForceUse(android_audio_legacy::AudioSystem::force_use, android_audio_legacy::AudioSystem::forced_config)':
12:55:13    ERROR -  hardware/libhardware_legacy/audio/AudioPolicyManagerBase.cpp:546:39: error: 'FFOS_AUDIO_DEVICE_OUT_FM' was not declared in this scope
12:55:13    ERROR -  hardware/libhardware_legacy/audio/AudioPolicyManagerBase.cpp:549:62: error: 'FFOS_AUDIO_FM_DEVICE_ADDRESS' was not declared in this scope
12:55:13    ERROR -  hardware/libhardware_legacy/audio/AudioPolicyManagerBase.cpp:552:64: error: 'FFOS_AUDIO_FM_DEVICE_ADDRESS' was not declared in this scope
(Assignee)

Updated

3 years ago
Flags: needinfo?(kli)
Can you './repo sync system/core' and then check again?
In the latest sourece the following 2 definitions are in "system/core/include/system/audio.h"

#define FFOS_AUDIO_FM_DEVICE_ADDRESS "0"
#define FFOS_AUDIO_DEVICE_OUT_FM     0X80000


Partner fixed FM issue in the following commit/repo.

system/core
commit dfc314f61b701c4a710b989e9b21300d122dbbc9
Author: cihong.liu <cihong.liu@spreadtrum.com>
Date:   Tue May 27 15:47:46 2014 +0800

hardware/libhardware_legacy
commit f19a0f6deb55d1040edbf68ddd6064c14515713d
Author: cihong.liu <cihong.liu@spreadtrum.com>
Date:   Tue May 27 15:40:45 2014 +0800
Flags: needinfo?(kli)
(Assignee)

Comment 11

3 years ago
Looks like it's not getting picked up in our mirror: 
http://git.mozilla.org/?p=external/sprd-aosp/platform/system/core.git;a=shortlog;h=refs/heads/sprdb2g_gonk4.4
(Assignee)

Updated

3 years ago
Depends on: 1017623
(Assignee)

Comment 12

3 years ago
Created attachment 8431045 [details] [diff] [review]
buildbot-configs for dolphin on b2g30-v1.4
Attachment #8431045 - Flags: review?(rail)
(Assignee)

Comment 13

3 years ago
Created attachment 8431080 [details] [diff] [review]
in-tree config.json for dolphin

still don't have a successful build yet, but this has been what I've used so far
Attachment #8431080 - Flags: review?(rail)
Thanks catlee - once again you covered for me being toooo slow... :-/
Attachment #8431045 - Flags: review?(rail) → review+
Attachment #8431080 - Flags: review?(rail) → review+
(Assignee)

Comment 15

3 years ago
builds look good now in staging. will deploy tomorrow

Updated

3 years ago
Assignee: sbruno → catlee
(Assignee)

Updated

3 years ago
Attachment #8431045 - Flags: checked-in+
(Assignee)

Updated

3 years ago
Attachment #8431080 - Flags: checked-in+
Merged to production and deployed.
Depends on: 1021437
Chris, 

I'd like to to confirm one thing. Dolphin need to be built kernelheader first as tarako.
But I am not sure if dophin build is done as the same as tarako.
$./build.sh kernelheader && ./build.sh

As I saw there is an error in bug 1022078, I think build kernelheader first can prevent this error.
Flags: needinfo?(catlee)
(Assignee)

Comment 18

3 years ago
If that's the case, then can you add "build_targets": ["kernelheader", ""], to https://hg.mozilla.org/releases/mozilla-b2g30_v1_4/file/51fed8981e43/b2g/config/dolphin/config.json#l6, like we have here: https://hg.mozilla.org/releases/mozilla-b2g28_v1_3t/file/default/b2g/config/tarako/config.json#l7
Flags: needinfo?(catlee)
Chris, Thanks!
(Assignee)

Comment 20

3 years ago
Looks like builds are green now.
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED

Comment 21

3 years ago
Two bugs still need your help.
 - [Bug 1027554] dolphin full image size is incorrect in pvt build
 - [Bug 1032066] [FxOS][V1.4][Dolphin] Default user of engineering build isn't "root"

Thanks! Have a nice day!
You need to log in before you can comment on or make changes to this bug.