Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in

Issue 11361103: msvs: set $(TargetExt) when the "product_extension" is set

7 years, 1 month ago by tootallnate
7 years ago
bradn, scottmg


msvs: set $(TargetExt) when the "product_extension" is set For node.js native modules, we set the "product_extension" to "node", so that the resulting loadable_module has a ".node" file extension. On Windows, doing this results in a msbuild warning along the lines of: warning MSB8012: TargetExt(.dll) does not match the Linker's OutputFile property value (.node). This may cause your project to build incorrectly. To correct this, please make sure that $(OutDir), $(TargetName) and $(TargetExt) property values match the value specified in %(Link.OutputFile). This patch properly sets the $(TargetExt) variable to fix the warning. BUG=

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M pylib/gyp/generator/ View 2 chunks +6 lines, -0 lines 0 comments Download


Total messages: 3 (0 generated)
M-A Ruel
1. You can't use the commit queue for GYP. 2. You need to click "Publish+Mail ...
7 years, 1 month ago (2012-11-07 13:52:00 UTC) #1
On 2012/11/07 13:52:00, Marc-Antoine Ruel wrote: > 1. You can't use the commit queue for ...
7 years ago (2012-11-08 20:04:26 UTC) #2
7 years ago (2012-11-16 22:55:45 UTC) #3
Adding scott.
A test of the changed behavior would be idea.

Powered by Google App Engine
This is Rietveld 408576698