Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(1)

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

Created:
5 years ago by tootallnate
Modified:
5 years ago
Reviewers:
bradn, scottmg
CC:
gyp-developer_googlegroups.com
Visibility:
Public.

Description

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/msvs.py View 2 chunks +6 lines, -0 lines 0 comments Download
Trybot results:

Messages

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 ...
5 years ago (2012-11-07 13:52:00 UTC) #1
tootallnate
On 2012/11/07 13:52:00, Marc-Antoine Ruel wrote: > 1. You can't use the commit queue for ...
5 years ago (2012-11-08 20:04:26 UTC) #2
bradn
5 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 efc10ee0f