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

Unified Diff: test/msvs/rules_stdout_stderr/rule_stderr.py

Issue 1148653011: msvs: Make sure stdout/stderr from rule commands get logged (Closed) Base URL: https://chromium.googlesource.com/external/gyp.git@master
Patch Set: Created 5 years, 7 months 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
Index: test/msvs/rules_stdout_stderr/rule_stderr.py
diff --git a/test/compiler-override/my_cxx.py b/test/msvs/rules_stdout_stderr/rule_stderr.py
old mode 100755
new mode 100644
similarity index 57%
copy from test/compiler-override/my_cxx.py
copy to test/msvs/rules_stdout_stderr/rule_stderr.py
index e2f0bdd51c7c55a4aadbb2206e4d5911b23b12a2..839666047cb45d9fa7a79ee7863c07873befbad9
--- a/test/compiler-override/my_cxx.py
+++ b/test/msvs/rules_stdout_stderr/rule_stderr.py
@@ -1,6 +1,7 @@
#!/usr/bin/env python
-# Copyright (c) 2012 Google Inc. All rights reserved.
+# Copyright (c) 2015 Google Inc. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
+
import sys
-print sys.argv
+print>>sys.stderr, "This will go to stderr"
scottmg 2015/05/28 15:18:51 nit; space before >>
« no previous file with comments | « test/msvs/rules_stdout_stderr/gyptest-rules-stdout-stderr.py ('k') | test/msvs/rules_stdout_stderr/rule_stdout.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698