From b41d5f1f40dfa9e77af69e1e9aa336d31754b9d2 Mon Sep 17 00:00:00 2001 From: Ilya Nikolaevskiy Date: Thu, 18 Jul 2019 09:59:25 +0200 Subject: [PATCH] Fix CE in rtp_generator. Add it to default build target This is a temporary solution, as there are several other executables and some tests in rtc_tools/BUILD.gn. Including all of them to default target is not decided yet. But as rtp_generator tends to be broken reguraly, It should be included there at least for now. Bug: webrtc:10807 Change-Id: I3acf5a93c74bf1e2474c6aaee35653efbb43d3a0 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/146080 Reviewed-by: Mirko Bonadei Commit-Queue: Ilya Nikolaevskiy Cr-Commit-Position: refs/heads/master@{#28595} --- BUILD.gn | 1 + rtc_tools/rtp_generator/rtp_generator.cc | 1 - 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/BUILD.gn b/BUILD.gn index af0de9f528..4a04def2c1 100644 --- a/BUILD.gn +++ b/BUILD.gn @@ -57,6 +57,7 @@ if (!build_with_chromium) { "modules/video_capture:video_capture_internal_impl", "pc:peerconnection_unittests", "pc:rtc_pc_unittests", + "rtc_tools:rtp_generator", "stats:rtc_stats_unittests", "system_wrappers:system_wrappers_unittests", "test", diff --git a/rtc_tools/rtp_generator/rtp_generator.cc b/rtc_tools/rtp_generator/rtp_generator.cc index 24528e8d4b..cd61d85617 100644 --- a/rtc_tools/rtp_generator/rtp_generator.cc +++ b/rtc_tools/rtp_generator/rtp_generator.cc @@ -178,7 +178,6 @@ RtpGenerator::RtpGenerator(const RtpGeneratorOptions& options) // Update some required to be unique values. stream_count++; video_config.rtp.mid = "mid-" + std::to_string(stream_count); - video_config.track_id = "track-" + std::to_string(stream_count); // Configure the video encoder configuration. VideoEncoderConfig encoder_config;