Skip to content

Conversation

tcare
Copy link
Contributor

@tcare tcare commented Sep 7, 2016

We'll use the Visual Studio 2015 platform toolset for now, this will just allow us to use the IDE without it complaining.

@tcare
Copy link
Contributor Author

tcare commented Sep 7, 2016

@curtisman or @digitalinfinity please review

@curtisman
Copy link
Contributor

Do we have problem compiling using v150 toolset?

@tcare
Copy link
Contributor Author

tcare commented Sep 7, 2016

__declspec(guard(overflow)) doesn't seem to work when I retarget, and it seems to retarget to v140 anyway. I don't see v150 installed. I didn't spend much time looking.

@@ -12,6 +12,7 @@
<PlatformToolset Condition="'$(VisualStudioVersion)'=='11.0'">v110</PlatformToolset>
<PlatformToolset Condition="'$(VisualStudioVersion)'=='12.0'">v120</PlatformToolset>
<PlatformToolset Condition="'$(VisualStudioVersion)'=='14.0'">v140</PlatformToolset>
<PlatformToolset Condition="'$(VisualStudioVersion)'=='15.0'">v140</PlatformToolset>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

v150?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe add a comment explaining why it doesn't match?

@tcare
Copy link
Contributor Author

tcare commented Sep 12, 2016

Ping. New projects in 15 Preview 4 use v140.

@dilijev
Copy link
Contributor

dilijev commented Sep 13, 2016

@tcare I still favor adding a comment to point out that the v140 isn't a typo. Otherwise LGTM

…io 2015 platform toolset for now, this will just allow us to use the IDE without it complaining.
@chakrabot chakrabot merged commit 77c3357 into chakra-core:master Sep 13, 2016
chakrabot pushed a commit that referenced this pull request Sep 13, 2016
Merge pull request #1553 from tcare:vs15

We'll use the Visual Studio 2015 platform toolset for now, this will just allow us to use the IDE without it complaining.
@tcare tcare deleted the vs15 branch September 13, 2016 01:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants