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

Delta Between Two Patch Sets: make_gyp_wrapper.py

Issue 29555620: Issue 5698 - Update V8 to 6.1.534.41 (Closed) Base URL: https://github.com/adblockplus/libadblockplus.git
Left Patch Set: Created Sept. 25, 2017, 7:46 a.m.
Right Patch Set: fix and rebase Created Sept. 28, 2017, 7:21 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Right: Side by side diff | Download
« dependencies ('K') | « dependencies ('k') | msvs_gyp_wrapper.py » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
(no file at all)
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # coding: utf-8 2 # coding: utf-8
3 """This script fixes the support of manually compiled static libraries for 3 """This script fixes the support of manually compiled static libraries for
4 android NDK build system. 4 android NDK build system.
5 5
6 Issue: 6 Issue:
7 Let's say there are some manually compiled libraries specified in 7 Let's say there are some manually compiled libraries specified in
8 link_settings.libraries or in ldflags of a gyp file. In this case ndk-build 8 link_settings.libraries or in ldflags of a gyp file. In this case ndk-build
9 passes them to a linker after system libraries, like c++_static, and 9 passes them to a linker after system libraries, like c++_static, and
10 as the result linker cannot find functions from system libraries. 10 as the result linker cannot find functions from system libraries.
(...skipping 101 matching lines...) Expand 10 before | Expand all | Expand 10 after
112 if variable == "LOCAL_STATIC_LIBRARIES": 112 if variable == "LOCAL_STATIC_LIBRARIES":
113 value_list.append("${ABP_STATIC_LIBRARIES_${BUILDTYPE}}") 113 value_list.append("${ABP_STATIC_LIBRARIES_${BUILDTYPE}}")
114 orig_WriteList(value_list, variable, prefix, quoter) 114 orig_WriteList(value_list, variable, prefix, quoter)
115 self.WriteList = types.MethodType(overridden_WriteList, self) 115 self.WriteList = types.MethodType(overridden_WriteList, self)
116 orig_MakefileWriter_WriteAndroidNdkModuleRule(self, module_name, all_sources , link_deps) 116 orig_MakefileWriter_WriteAndroidNdkModuleRule(self, module_name, all_sources , link_deps)
117 self.WriteList = orig_WriteList 117 self.WriteList = orig_WriteList
118 118
119 MakefileWriter.Write = overridden_Write 119 MakefileWriter.Write = overridden_Write
120 MakefileWriter.WriteAndroidNdkModuleRule = overridden_WriteAndroidNdkModuleRule 120 MakefileWriter.WriteAndroidNdkModuleRule = overridden_WriteAndroidNdkModuleRule
121 121
122 # Issue 5393
123 # replace $(LD_INPUTS) by "-Wl,--start-group $(LD_INPUTS) -Wl,--end-group" but
124 # only for cmd_link_host and only on linux.
125 print platform.system()
126 if platform.system() == "Linux":
127 gyp.generator.make.LINK_COMMANDS_ANDROID = \
128 gyp.generator.make.LINK_COMMANDS_ANDROID[:663] + \
129 "-Wl,--start-group $(LD_INPUTS) -Wl,--end-group" + \
130 gyp.generator.make.LINK_COMMANDS_ANDROID[675:]
131
132 if __name__ == '__main__': 122 if __name__ == '__main__':
133 gyp.main(sys.argv[1:]) 123 gyp.main(sys.argv[1:])
LEFTRIGHT

Powered by Google App Engine
This is Rietveld