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

Delta Between Two Patch Sets: adblockplus.gyp

Issue 29324576: Issue 2599, 2940 - Fix the presence of correct ATL version (Closed)
Left Patch Set: address comments Created Oct. 21, 2015, 11:52 a.m.
Right Patch Set: address comment, precise python version Created Dec. 2, 2015, 9:11 p.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 | « README.md ('k') | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 { 1 {
2 'includes': [ 2 'includes': [
3 'defaults.gypi', 3 'defaults.gypi',
4 'common/common.gypi', 4 'common/common.gypi',
5 ], 5 ],
6 6
7 'variables': { 7 'variables': {
8 'build_type%': 'devbuild', 8 'build_type%': 'devbuild',
9 'build_version%': '', 9 'build_version%': '',
10 }, 10 },
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 'src/shared/CriticalSection.h', 47 'src/shared/CriticalSection.h',
48 'src/shared/Dictionary.cpp', 48 'src/shared/Dictionary.cpp',
49 'src/shared/Dictionary.h', 49 'src/shared/Dictionary.h',
50 'src/shared/Utils.cpp', 50 'src/shared/Utils.cpp',
51 'src/shared/Utils.h', 51 'src/shared/Utils.h',
52 'src/shared/Version.h', 52 'src/shared/Version.h',
53 'src/shared/MsHTMLUtils.cpp', 53 'src/shared/MsHTMLUtils.cpp',
54 'src/shared/MsHTMLUtils.h', 54 'src/shared/MsHTMLUtils.h',
55 ], 55 ],
56 'include_dirs': [ 56 'include_dirs': [
57 '$(ATLDir)/include', 57 '$(ADBLOCKPLUS_ATL)/include',
58 ], 58 ],
59 }, 59 },
60 60
61 { 61 {
62 'target_name': 'AdblockPlusEngine', 62 'target_name': 'AdblockPlusEngine',
63 'type': 'executable', 63 'type': 'executable',
64 'dependencies': [ 64 'dependencies': [
65 'shared', 65 'shared',
66 'libadblockplus/libadblockplus.gyp:libadblockplus', 66 'libadblockplus/libadblockplus.gyp:libadblockplus',
67 ], 67 ],
68 'sources': [ 68 'sources': [
69 'src/engine/Main.cpp', 69 'src/engine/Main.cpp',
70 'src/engine/NotificationWindow.cpp', 70 'src/engine/NotificationWindow.cpp',
71 'src/engine/NotificationWindow.h', 71 'src/engine/NotificationWindow.h',
72 'src/engine/Debug.cpp', 72 'src/engine/Debug.cpp',
73 'src/engine/UpdateInstallDialog.cpp', 73 'src/engine/UpdateInstallDialog.cpp',
74 'src/engine/Updater.cpp', 74 'src/engine/Updater.cpp',
75 'src/engine/engine.rc', 75 'src/engine/engine.rc',
76 ], 76 ],
77 'include_dirs': [ 77 'include_dirs': [
78 '$(ATLDir)/include', 78 '$(ADBLOCKPLUS_ATL)/include',
79 ], 79 ],
80 'libraries': [ 80 'libraries': [
81 '-ladvapi32', 81 '-ladvapi32',
82 '-lole32', 82 '-lole32',
83 '-luser32', 83 '-luser32',
84 '-lshell32', 84 '-lshell32',
85 '-lshlwapi', 85 '-lshlwapi',
86 ], 86 ],
87 'msbuild_toolset': 'v110_xp', 87 'msbuild_toolset': 'v110_xp',
88 'msvs_settings': { 88 'msvs_settings': {
89 'VCLinkerTool': { 89 'VCLinkerTool': {
90 'DelayLoadDLLs': ['Shell32.dll'], 90 'DelayLoadDLLs': ['Shell32.dll'],
91 'conditions': [[ 91 'conditions': [[
92 'target_arch=="ia32"', { 92 'target_arch=="ia32"', {
93 'AdditionalLibraryDirectories': [ 93 'AdditionalLibraryDirectories': [
94 '$(ATLDir)/lib', 94 '$(ADBLOCKPLUS_ATL)/lib',
95 ], 95 ],
96 }, { 96 }, {
97 'AdditionalLibraryDirectories': [ 97 'AdditionalLibraryDirectories': [
98 '$(ATLDir)/lib/amd64', 98 '$(ADBLOCKPLUS_ATL)/lib/amd64',
99 ], 99 ],
100 } 100 }
101 ]], 101 ]],
102 }, 102 },
103 }, 103 },
104 }, 104 },
105 105
106 { 106 {
107 'target_name': 'AdblockPlus', 107 'target_name': 'AdblockPlus',
108 'type': 'shared_library', 108 'type': 'shared_library',
(...skipping 28 matching lines...) Expand all
137 'src/plugin/PluginDebug.cpp', 137 'src/plugin/PluginDebug.cpp',
138 'src/plugin/PluginDebug.h', 138 'src/plugin/PluginDebug.h',
139 'src/plugin/PluginDomTraverserBase.h', 139 'src/plugin/PluginDomTraverserBase.h',
140 'src/plugin/PluginErrorCodes.h', 140 'src/plugin/PluginErrorCodes.h',
141 'src/plugin/PluginFilter.cpp', 141 'src/plugin/PluginFilter.cpp',
142 'src/plugin/PluginFilter.h', 142 'src/plugin/PluginFilter.h',
143 'src/plugin/PluginMimeFilterClient.cpp', 143 'src/plugin/PluginMimeFilterClient.cpp',
144 'src/plugin/PluginMimeFilterClient.h', 144 'src/plugin/PluginMimeFilterClient.h',
145 'src/plugin/PluginMutex.cpp', 145 'src/plugin/PluginMutex.cpp',
146 'src/plugin/PluginMutex.h', 146 'src/plugin/PluginMutex.h',
147 'src/plugin/PluginPassthroughObject.h',
148 'src/plugin/PluginSettings.cpp', 147 'src/plugin/PluginSettings.cpp',
149 'src/plugin/PluginSettings.h', 148 'src/plugin/PluginSettings.h',
150 'src/plugin/PluginStdAfx.cpp', 149 'src/plugin/PluginStdAfx.cpp',
151 'src/plugin/PluginStdAfx.h', 150 'src/plugin/PluginStdAfx.h',
152 'src/plugin/PluginSystem.cpp', 151 'src/plugin/PluginSystem.cpp',
153 'src/plugin/PluginSystem.h', 152 'src/plugin/PluginSystem.h',
154 'src/plugin/PluginTabBase.cpp', 153 'src/plugin/PluginTabBase.cpp',
155 'src/plugin/PluginTabBase.h', 154 'src/plugin/PluginTabBase.h',
156 'src/plugin/PluginUserSettings.cpp', 155 'src/plugin/PluginUserSettings.cpp',
157 'src/plugin/PluginUserSettings.h', 156 'src/plugin/PluginUserSettings.h',
158 'src/plugin/PluginUtil.cpp', 157 'src/plugin/PluginUtil.cpp',
159 'src/plugin/PluginUtil.h', 158 'src/plugin/PluginUtil.h',
160 'src/plugin/PluginWbPassThrough.cpp', 159 'src/plugin/PluginWbPassThrough.cpp',
161 'src/plugin/PluginWbPassThrough.h', 160 'src/plugin/PluginWbPassThrough.h',
162 'src/plugin/ProtocolCF.h',
163 'src/plugin/ProtocolCF.inl',
164 'src/plugin/ProtocolImpl.h',
165 'src/plugin/ProtocolImpl.inl',
166 'src/plugin/Resource.h', 161 'src/plugin/Resource.h',
167 'src/plugin/SinkPolicy.h', 162 'src/plugin/passthroughapp/PassthroughObject.h',
168 'src/plugin/SinkPolicy.inl', 163 'src/plugin/passthroughapp/ProtocolCF.h',
169 ], 164 'src/plugin/passthroughapp/ProtocolCF.inl',
170 'include_dirs': [ 165 'src/plugin/passthroughapp/ProtocolImpl.h',
171 '$(ATLDir)/include', 166 'src/plugin/passthroughapp/ProtocolImpl.inl',
167 'src/plugin/passthroughapp/SinkPolicy.h',
168 'src/plugin/passthroughapp/SinkPolicy.inl',
169 ],
170 'include_dirs': [
171 '$(ADBLOCKPLUS_ATL)/include',
172 ], 172 ],
173 # See "Adding Visual Style Support to an Extension, Plug-in, MMC Snap-in or a DLL 173 # See "Adding Visual Style Support to an Extension, Plug-in, MMC Snap-in or a DLL
174 # That Is Brought into a Process" on the link here: 174 # That Is Brought into a Process" on the link here:
175 # http://msdn.microsoft.com/en-us/library/windows/desktop/bb773175%28v=vs.85 %29.aspx#using_manifests 175 # http://msdn.microsoft.com/en-us/library/windows/desktop/bb773175%28v=vs.85 %29.aspx#using_manifests
176 'defines': ['ISOLATION_AWARE_ENABLED'], 176 'defines': ['ISOLATION_AWARE_ENABLED'],
177 'libraries': [ 177 'libraries': [
178 '-lwinhttp', 178 '-lwinhttp',
179 '-lshell32', 179 '-lshell32',
180 '-lComctl32', 180 '-lComctl32',
181 '-lGdi32', 181 '-lGdi32',
182 ], 182 ],
183 'msvs_settings': { 183 'msvs_settings': {
184 'VCLinkerTool': { 184 'VCLinkerTool': {
185 'conditions': [[ 185 'conditions': [[
186 'target_arch=="ia32"', { 186 'target_arch=="ia32"', {
187 'AdditionalLibraryDirectories': [ 187 'AdditionalLibraryDirectories': [
188 '$(ATLDir)/lib', 188 '$(ADBLOCKPLUS_ATL)/lib',
189 ], 189 ],
190 }, { 190 }, {
191 'AdditionalLibraryDirectories': [ 191 'AdditionalLibraryDirectories': [
192 '$(ATLDir)/lib/amd64', 192 '$(ADBLOCKPLUS_ATL)/lib/amd64',
193 ], 193 ],
194 } 194 }
195 ]], 195 ]],
196 'DelayLoadDLLs': ['Shell32.dll'], 196 'DelayLoadDLLs': ['Shell32.dll'],
197 }, 197 },
198 }, 198 },
199 }, 199 },
200 200
201 { 201 {
202 'target_name': 'tests', 202 'target_name': 'tests',
(...skipping 26 matching lines...) Expand all
229 'type': 'executable', 229 'type': 'executable',
230 'dependencies': [ 230 'dependencies': [
231 'shared', 231 'shared',
232 'libadblockplus/libadblockplus.gyp:libadblockplus', 232 'libadblockplus/libadblockplus.gyp:libadblockplus',
233 'libadblockplus/third_party/googletest.gyp:googletest_main', 233 'libadblockplus/third_party/googletest.gyp:googletest_main',
234 ], 234 ],
235 'sources': [ 235 'sources': [
236 'src/plugin/PluginUserSettings.cpp', 236 'src/plugin/PluginUserSettings.cpp',
237 'src/plugin/PluginUserSettings.h', 237 'src/plugin/PluginUserSettings.h',
238 'test/plugin/UserSettingsTest.cpp', 238 'test/plugin/UserSettingsTest.cpp',
239 'src/plugin/PluginUtil.h',
240 'test/plugin/UtilTest.cpp',
239 # 241 #
240 # required only for linking 242 # required only for linking
241 # 243 #
242 'src/plugin/AdblockPlusClient.cpp', 244 'src/plugin/AdblockPlusClient.cpp',
243 'src/plugin/AdblockPlusDomTraverser.cpp', 245 'src/plugin/AdblockPlusDomTraverser.cpp',
244 'src/plugin/ATL_Deprecate.cpp', 246 'src/plugin/ATL_Deprecate.cpp',
245 'src/plugin/NotificationMessage.cpp', 247 'src/plugin/NotificationMessage.cpp',
246 'src/plugin/Plugin.cpp', 248 'src/plugin/Plugin.cpp',
247 'src/plugin/PluginClientBase.cpp', 249 'src/plugin/PluginClientBase.cpp',
248 'src/plugin/PluginClientFactory.cpp', 250 'src/plugin/PluginClientFactory.cpp',
249 'src/plugin/PluginClass.cpp', 251 'src/plugin/PluginClass.cpp',
250 'src/plugin/PluginDebug.cpp', 252 'src/plugin/PluginDebug.cpp',
251 'src/plugin/PluginFilter.cpp', 253 'src/plugin/PluginFilter.cpp',
252 'src/plugin/PluginMimeFilterClient.cpp', 254 'src/plugin/PluginMimeFilterClient.cpp',
253 'src/plugin/PluginMutex.cpp', 255 'src/plugin/PluginMutex.cpp',
254 'src/plugin/PluginSettings.cpp', 256 'src/plugin/PluginSettings.cpp',
255 'src/plugin/PluginSystem.cpp', 257 'src/plugin/PluginSystem.cpp',
256 'src/plugin/PluginTabBase.cpp', 258 'src/plugin/PluginTabBase.cpp',
257 'src/plugin/PluginUtil.cpp', 259 'src/plugin/PluginUtil.cpp',
258 'src/plugin/PluginWbPassthrough.cpp', 260 'src/plugin/PluginWbPassthrough.cpp',
259 ], 261 ],
260 'include_dirs': [ 262 'include_dirs': [
261 '$(ATLDir)/include', 263 '$(ADBLOCKPLUS_ATL)/include',
262 ], 264 ],
263 'defines': [ 265 'defines': [
264 'WINVER=0x0501', 266 'WINVER=0x0501',
265 'PRODUCT_ADBLOCKPLUS' 267 'PRODUCT_ADBLOCKPLUS'
266 ], 268 ],
267 'link_settings': { 269 'link_settings': {
268 'libraries': ['-ladvapi32', '-lshell32', '-lole32', '-lComctl32', '-lGdi32 '], 270 'libraries': ['-ladvapi32', '-lshell32', '-lole32', '-lComctl32', '-lGdi32 '],
269 }, 271 },
270 'msvs_settings': { 272 'msvs_settings': {
271 'VCLinkerTool': { 273 'VCLinkerTool': {
272 'SubSystem': '1', # Console 274 'SubSystem': '1', # Console
273 'EntryPointSymbol': 'mainCRTStartup', 275 'EntryPointSymbol': 'mainCRTStartup',
274 'conditions': [[ 276 'conditions': [[
275 'target_arch=="ia32"', { 277 'target_arch=="ia32"', {
276 'AdditionalLibraryDirectories': [ 278 'AdditionalLibraryDirectories': [
277 '$(ATLDir)/lib', 279 '$(ADBLOCKPLUS_ATL)/lib',
278 ], 280 ],
279 }, { 281 }, {
280 'AdditionalLibraryDirectories': [ 282 'AdditionalLibraryDirectories': [
281 '$(ATLDir)/lib/amd64', 283 '$(ADBLOCKPLUS_ATL)/lib/amd64',
282 ], 284 ],
283 } 285 }
284 ]], 286 ]],
285 }, 287 },
286 }, 288 },
287 }] 289 }]
288 } 290 }
LEFTRIGHT
« README.md ('k') | no next file » | Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Toggle Comments ('s')

Powered by Google App Engine
This is Rietveld