1903952 - add gtest build in loop-ff.sh

Status: RESOLVED FIXED

(Core :: WebRTC, task)

Description

[Michael Froman [:mjf]]

Without building the gtest code we can miss failures that show up later on try builds, specifically for api changes that affect media/webrtc/signaling/gtest/MockCall.h. Adding ./mach build recurse_gtest after doing ./mach build adds a little time but avoids having to back-track and find where a simple fix to non-libwebrtc code is required.

Comment 1

[Michael Froman [:mjf]]

Attachment: Bug 1903952 - add './mach build recurse_gtest' to sanity builds. r?ng!

Adding this to the build step after each new commit will help
uncover errors that we miss until try builds happen. Specifically,
this shows build errors in media/webrtc/signaling/gtest/MockCall.h
resulting from upstream api changes.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e078fa0b1691
add './mach build recurse_gtest' to sanity builds. r=ng DONTBUILD

Comment 3

[Cristina Horotan [:chorotan]]

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