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

Unified Diff: test/variables/commands/commands.gyp

Issue 1454433002: Python 3 compatibility Base URL: https://chromium.googlesource.com/external/gyp.git@master
Patch Set: Rebase with master (4ec6c4e3a94bd04a6da2858163d40b2429b8aad1) Created 4 years, 8 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/variables/commands/commands.gyp
diff --git a/test/variables/commands/commands.gyp b/test/variables/commands/commands.gyp
index 78376edd9959cb3ecf69e2609b60a0050975e012..7d460b1810f6ab0ae2fd7e502e1285fc8626e01e 100644
--- a/test/variables/commands/commands.gyp
+++ b/test/variables/commands/commands.gyp
@@ -8,7 +8,7 @@
{
'variables': {
- 'pi': 'import math; print math.pi',
+ 'pi': 'import math; print(math.pi)',
Nico 2016/07/29 22:22:06 hmm doesn't this need an `from __future__ import p
AWhetter 2016/11/05 23:59:50 Nope. Using brackets is optional in Python 2.7.
'third_letters': "<(other_letters)HIJK",
'letters_list': 'ABCD',
'other_letters': '<(letters_list)EFG',
@@ -41,9 +41,9 @@
'type': 'none',
'variables': {
'var1': '<!(["python", "-c", "<(pi)"])',
- 'var2': '<!(python -c "print \'<!(python -c "<(pi)") <(letters_list)\'")',
- 'var3': '<!(python -c "print \'<(letters_list)\'")',
- 'var4': '<(<!(python -c "print \'letters_list\'"))',
+ 'var2': '<!(python -c "print(\'<!(python -c "<(pi)") <(letters_list)\')")',
+ 'var3': '<!(python -c "print(\'<(letters_list)\')")',
+ 'var4': '<(<!(python -c "print(\'letters_list\')"))',
'var5': 'letters_',
'var6': 'list',
'var7': '<(check_int)',

Powered by Google App Engine
This is Rietveld 408576698