Last known good revision

129 views
Skip to first unread message

EARob Agar

unread,
Oct 20, 2016, 7:28:01 AM10/20/16
to webrtcbuilds
Building against the latest commit seems dangerous. Is there a good way to determine the latest stable revision?

Alexandre GOUAILLARD

unread,
Oct 20, 2016, 8:42:27 AM10/20/16
to EARob Agar, webrtcbuilds
Use the google release branches.

Sent from my iPhone

On 20 Oct 2016, at 18:28, EARob Agar <ea.ro...@gmail.com> wrote:

Building against the latest commit seems dangerous. Is there a good way to determine the latest stable revision?

--
You received this message because you are subscribed to the Google Groups "webrtcbuilds" group.
To unsubscribe from this group and stop receiving emails from it, send an email to webrtcbuilds...@googlegroups.com.
To post to this group, send email to webrtc...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/webrtcbuilds/9c60bf55-822d-4080-a4db-0b2d6176ba38%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

EARob Agar

unread,
Oct 20, 2016, 10:04:41 AM10/20/16
to webrtcbuilds
Ah, so build.sh -b branch-heads/54 will build WebRTC as used in Chromium 54?

Shaul Fridman

unread,
Nov 10, 2016, 7:44:35 PM11/10/16
to webrtcbuilds
Did you manage to build the webrtc library?
I tried it several times, and it always fail, each time from a different reason.
?(If you did mange to create the library I will be ver thankful if you can share the build result (binaries + h files, I assume 


בתאריך יום חמישי, 20 באוקטובר 2016 בשעה 17:04:41 UTC+3, מאת EARob Agar:

EARob Agar

unread,
Nov 17, 2016, 5:31:41 AM11/17/16
to webrtcbuilds
I did, but not using any of the Chrome release branches as we needed a more recent fix.  Can't use the master head right now due to linker errors - see #41 for details. The last known good revision is 3355f6d.

./build.sh -r 3355f6d

Alexander Widerberg

unread,
Feb 8, 2017, 5:16:53 AM2/8/17
to webrtcbuilds
I can confirm that branch-heads/55 compiles on OS X at least (with minor modifications on a separate fork of the repo). 
Reply all
Reply to author
Forward
0 new messages