Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: msvs_gyp_wrapper.py

Issue 29451722: Issue 4907 - Update v8 to 5.7.278 in libadblockplus (Closed) Base URL: https://github.com/adblockplus/libadblockplus.git
Left Patch Set: Created May 30, 2017, 3:02 p.m.
Right Patch Set: independence from file mode of make_gyp_wrapper.py Created July 4, 2017, 11:11 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « make_gyp_wrapper.py ('k') | shell/shell.gyp » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # coding: utf-8 2 # coding: utf-8
3 3
4 import os 4 import os
5 import sys 5 import sys
6 6
7 base_dir = os.path.abspath(os.path.dirname(__file__)) 7 base_dir = os.path.abspath(os.path.dirname(__file__))
8 sys.path.append(os.path.join(base_dir, 'third_party', 'gyp', 'pylib')) 8 sys.path.append(os.path.join(base_dir, 'third_party', 'gyp', 'pylib'))
9 import gyp 9 import gyp
10 import gyp.generator.msvs 10 import gyp.generator.msvs
11 11
12 orig_fix_path = gyp.generator.msvs._FixPath 12 orig_fix_path = gyp.generator.msvs._FixPath
13 13
14 # gyp is trying to expand parameters as paths and the fix prevents gyp from
15 # doing it for particular parameters for particular tools.
16 # Don't touch following js2c and build-v8 parameters
14 dont_expand = [ 17 dont_expand = [
15 # js2c 18 # js2c
Eric 2017/05/30 17:31:41 # js2c paths
16 'CORE', 'EXPERIMENTAL', 'off', 'EXTRAS', 'EXPERIMENTAL_EXTRAS', 19 'CORE', 'EXPERIMENTAL', 'off', 'EXTRAS', 'EXPERIMENTAL_EXTRAS',
17 # build-v8 20 # build-v8
Eric 2017/05/30 17:31:41 # build-v8 paths The new comments only made sense
sergei 2017/05/31 12:47:12 they are not paths, they are parameters for js2c a
18 'ia32', 'x64' 21 'ia32', 'x64'
19 ] 22 ]
20 23
21 def _FixPath(path): 24 def _FixPath(path):
22 if path in dont_expand: 25 if path in dont_expand:
23 # Don't touch js2c and build-v8 parameters
24 return path 26 return path
25 return orig_fix_path(path) 27 return orig_fix_path(path)
26 gyp.generator.msvs._FixPath = _FixPath 28 gyp.generator.msvs._FixPath = _FixPath
27 29
28 if __name__ == '__main__': 30 if __name__ == '__main__':
29 gyp.main(sys.argv[1:]) 31 gyp.main(sys.argv[1:])
LEFTRIGHT

Powered by Google App Engine
This is Rietveld