1142494 - Don't try to package the steeplechase tests if webrtc is disabled

Status: RESOLVED FIXED

(Core :: WebRTC, defect, P2)

Description

[Ryan VanderMeulen [:RyanVM]]

We were hitting bustage on Emulator L builds during packaging because webrtc was disabled and it was trying to package the steeplechase tests. We shouldn't try to do so unless webrtc is enabled.

Comment 1

[Ryan VanderMeulen [:RyanVM]]

Attachment: patch

Try confirms that this works as expected. Nils, speak now or forever hold your peace if there's any reason to not do this :)

Comment 2

[Nils Ohlmeier [:drno]]

Comment on attachment 8576696 [details] [diff] [review]
patch

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

If I understand the idea correct this only means: don't packages steeplechase (plus its tests) if MOZ_WEBRTC is not defined. That sounds perfectly reasonable.

My only question/concern is: does this affect the *tests.zip file we publish together with the binary tarballs?
Because our current steeplechase setup in the MV QA lab extracts steeplechase from these test zip files. So excluding steeplechase from that would cause major trouble for us (my + assumes that your patch is not going that far :-) ).

Comment 3

[Ryan VanderMeulen [:RyanVM]]

I don't see how it would affect tests.zip except for platforms where packaging them would have been useless anyway (i.e. they wouldn't be able to run on a webrtc-disabled build anyway). Below are my Try pushes if you want to take a look at the packages and verify, though.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=a65bea9f145a
https://treeherder.mozilla.org/#/jobs?repo=try&revision=721313d119fd

Comment 4

[Ryan VanderMeulen [:RyanVM]]

Thanks :)
https://hg.mozilla.org/integration/b2g-inbound/rev/b3ae8d7b0426
https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/b9c4d7f2994b

Comment 5

[Ryan VanderMeulen [:RyanVM]]

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

Comment 6

[Ryan VanderMeulen [:RyanVM]]

Fix for accidentally breaking OSX test packaging (whoopsie!):
https://hg.mozilla.org/mozilla-central/rev/62395f12dbba
https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/3db27b3c9298