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.
Created attachment 8576696 [details] [diff] [review] 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 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 :-) ).
Attachment #8576696 - Flags: feedback?(drno) → feedback+
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
Priority: -- → P2
Attachment #8576696 - Flags: review?(gps) → review+
Thanks :) https://hg.mozilla.org/integration/b2g-inbound/rev/b3ae8d7b0426 https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/b9c4d7f2994b
status-b2g-v2.2: --- → fixed
status-b2g-master: --- → fixed
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
status-firefox39: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
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
You need to log in before you can comment on or make changes to this bug.