1203144 - Install tooltool packages for emulator builds

Status: RESOLVED FIXED

(Taskcluster :: General, defect, P1)

Description

[Wander Lairson Costa]

We should run install-packages.sh for emulator builds as we do for b2g desktop.

Comment 1

[Wander Lairson Costa]

Attachment: Add tooltool dependencies to emulator builds. r=garndt,dustin

Comment 2

[Wander Lairson Costa]

Comment on attachment 8658781 [details] [diff] [review]
Add tooltool dependencies to emulator builds. r=garndt,dustin

https://tools.taskcluster.net/task-graph-inspector/#Gem1lePkSCav66cEEPDg9w/My-0gYPjQZqAYdMz-SvD3A/

Comment 3

[Wander Lairson Costa]

Tim, could you please check the problems with gtk3 are gone?

Comment 4

[Ricky Chien [:rickychien] (inactive)]

Thank you wcosta!

Here is my question:

LD_LIBRARY_PATH doesn't append the path of gtk3 in b2g-emulator-kk, see 

https://s3-us-west-2.amazonaws.com/taskcluster-public-artifacts/MLfcrQRHQKu4n1bie42FIA/0/public/logs/live_backing.log

I think it should point to gtk3 like b2g-desktop-build:

https://bugzilla.mozilla.org/show_bug.cgi?id=1146713#c118

Is that ok to append gtk3 in LD_LIBRARY_PATH within this bug? thanks!

Comment 5

[Dustin J. Mitchell [:dustin] (he/him)]

Comment on attachment 8658781 [details] [diff] [review]
Add tooltool dependencies to emulator builds. r=garndt,dustin

Review of attachment 8658781 [details] [diff] [review]:
-----------------------------------------------------------------

I'm not terribly familiar with these scripts, but everything I can see here looks right :)

Comment 6

[Wander Lairson Costa]

(In reply to Ricky Chien [:rickychien] from comment #4)
> Thank you wcosta!
> 
> Here is my question:
> 
> LD_LIBRARY_PATH doesn't append the path of gtk3 in b2g-emulator-kk, see 
> 
> https://s3-us-west-2.amazonaws.com/taskcluster-public-artifacts/
> MLfcrQRHQKu4n1bie42FIA/0/public/logs/live_backing.log
> 
> I think it should point to gtk3 like b2g-desktop-build:
> 
> https://bugzilla.mozilla.org/show_bug.cgi?id=1146713#c118
> 
> Is that ok to append gtk3 in LD_LIBRARY_PATH within this bug? thanks!

Hi Tim,

It feels like this is more a emulator/gecko build issue than Taskcluster. Tracking down the desktop build, it adds [0], which sets up gtk. I don't the inner works of B2G build system to say how things should work for emulator builds.

[0] https://dxr.mozilla.org/mozilla-central/source/build/unix/mozconfig.gtk

Comment 7

[Greg Arndt [:garndt]]

Comment on attachment 8658781 [details] [diff] [review]
Add tooltool dependencies to emulator builds. r=garndt,dustin

Review of attachment 8658781 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM

Comment 8

[Ricky Chien [:rickychien] (inactive)]

(In reply to Wander Lairson Costa [:wcosta] from comment #6)
> (In reply to Ricky Chien [:rickychien] from comment #4)
> > Thank you wcosta!
> > 
> > Here is my question:
> > 
> > LD_LIBRARY_PATH doesn't append the path of gtk3 in b2g-emulator-kk, see 
> > 
> > https://s3-us-west-2.amazonaws.com/taskcluster-public-artifacts/
> > MLfcrQRHQKu4n1bie42FIA/0/public/logs/live_backing.log
> > 
> > I think it should point to gtk3 like b2g-desktop-build:
> > 
> > https://bugzilla.mozilla.org/show_bug.cgi?id=1146713#c118
> > 
> > Is that ok to append gtk3 in LD_LIBRARY_PATH within this bug? thanks!
> 
> Hi Tim,
> 
> It feels like this is more a emulator/gecko build issue than Taskcluster.
> Tracking down the desktop build, it adds [0], which sets up gtk. I don't the
> inner works of B2G build system to say how things should work for emulator
> builds.

It's fine! I just don't know gtk3 path is set by taskcluster or b2g / gecko build.

Again, thanks for your help!

Comment 9

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Comment on attachment 8658781 [details] [diff] [review]
Add tooltool dependencies to emulator builds. r=garndt,dustin

Per comment 8.

Comment 10

[Tim Guan-tin Chien [:timdream] (please needinfo)]

I guess we could check this in?

Comment 11

[Pulsebot]

https://hg.mozilla.org/integration/b2g-inbound/rev/be57887bca91

Comment 12

[Phil Ringnalda (:philor)]

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