820203 - Add Marionette to the package produced by ./build.sh package-tests

Status: RESOLVED FIXED

(Firefox OS Graveyard :: GonkIntegration, defect)

Description

[Jonathan Griffin (:jgriffin)]

This will be the best way to get Marionette installed for running Gaia tests in buildbot.

Comment 1

[Jonathan Griffin (:jgriffin)]

mwu, since gecko has an empty Android.mk, how would you like me to do this?  Add a non-empty Android.mk just for this, or just put the logic in gonk-misc/Android.mk?

Comment 2

[Michael Wu [:mwu]]

gonk-misc/Android.mk is appropriate. The gecko Android.mk is just to prevent the Android build system from looking into that directory tree and picking up Android.mks that it shouldn't be.

Comment 3

[Jonathan Griffin (:jgriffin)]

The disadvantage of using Gecko's Android.mk is that it won't respect custom GECKO_PATH.  This wouldn't be a problem for buildbot, I don't think, but it may be confusing to people who try to run this locally.

Comment 4

[Jonathan Griffin (:jgriffin)]

Attachment: Pointer to pull request

Comment 5

[Michael Wu [:mwu]]

Comment on attachment 690972 [details]
Pointer to pull request

merged

Comment 6

[Armen [:armenzg]]

Time for a new gonk-misc snapshot?

Comment 7

[Jonathan Griffin (:jgriffin)]

https://github.com/mozilla-b2g/gonk-misc/commit/7c43e273991a8a66afc88c888befb2830b7640e1

Armen, yes you'll need a new gonk-misc snapshot.