Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(265)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: third_party/gtest-parallel/README.webrtc

Issue 2513073002: Update gtest-parallel and introduce gtest-parallel-wrapper. (Closed)
Patch Set: Updated README.webrtc. Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | third_party/gtest-parallel/gtest-parallel » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/gtest-parallel/README.webrtc
diff --git a/third_party/gtest-parallel/README.webrtc b/third_party/gtest-parallel/README.webrtc
index 4d47151087bba541985a0ac4a2d395a48780706b..1f44e5e0f11eb0160bc2d8f49258b6e9296344dc 100644
--- a/third_party/gtest-parallel/README.webrtc
+++ b/third_party/gtest-parallel/README.webrtc
@@ -1,9 +1,13 @@
URL: https://github.com/google/gtest-parallel
-Version: d0cebaba01c5dbf3af8a1c89b64eed7596c2b56c
+Version: 89b859c1412089468599a3366a72a1e774ce5c2c
License: Apache 2.0
License File: LICENSE
Description:
Parallelization script for gtest binaries.
-Local Modifications: None
+Local Modifications:
+Added gtest-parallel-wrapper.py to convert the GTEST_SHARD_INDEX and
+GTEST_TOTAL_SHARDS environment variables to flags, since gtest-parallel takes
+flags as arguments, and delete them afterwards, since letting them set
+introduces a bug where only tests in the first shard are executed.
« no previous file with comments | « no previous file | third_party/gtest-parallel/gtest-parallel » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698