The key.snk is generated when you build, the problem is the csproj file hardcodes the directory to the sdk instead of using the appropriate project system variable like $(SDKToolsDir)<br><div class="gmail_quote"><div dir="ltr">On Thu, Aug 18, 2016 at 7:09 PM Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Llvm doesn't support vs2012 anymore, as long as it supports vs2013 it's fine <br><div class="gmail_quote"><div dir="ltr">On Thu, Aug 18, 2016 at 7:07 PM Antonio Maiorano <<a href="mailto:amaiorano@gmail.com" target="_blank">amaiorano@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>What I meant by upgrade was simply making it build in VS 2015. However, you bring up a valid point about making sure the extension will continue to work in VS 2012. I will look into that. Like those references that go from 10 to 14 that point out; I wonder if instead I should be able to bring in those version 10 assemblies via NuGet. I'll take a closer look.</div><div><br></div><div>Part of my change, however, seems to imply that the extension (vsix) project would not build correctly even in VS 2012. For instance, the missing Key.snk file. I don't have VS 2012 installed at the moment, so I cannot validate.</div><div><br></div><div>Thanks,</div><div><br></div><div>Antonio</div><div><div><br></div><div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, 18 Aug 2016 at 19:38 Hans Wennborg <<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Antonio,<br>
<br>
On Wed, Aug 17, 2016 at 8:15 AM, Antonio Maiorano via cfe-commits<br>
<<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>> wrote:<br>
> This patch for clang-format-vs includes the following:<br>
><br>
> - Upgrade to VS 2015, including .NET framework upgrade from 4.0 to 4.5, and<br>
> upgrading Microsoft.VisualStudio references to v14 versions<br>
> - Fix build by removing dependency on "Key.snk" file which was never checked<br>
> in (and not really required anyway)<br>
> - Add ".vs" directory to svn ignore (new folder that VS 2015 creates for<br>
> user settings)<br>
<br>
"What does "Upgrade to VS 2015 mean? Adding support for running the<br>
plugin in VS2015, or does it mean requiring VS2015 for building?<br>
<br>
+zturner: I thought the plugin already worked in VS 2015?<br>
<br>
I mostly just build the plugin without knowing exactly how this stuff<br>
works, but looking at the patch I'm worried that you're increasing the<br>
required version for building it? I see a bunch of values going from<br>
10 (VS 2012) to 14 (VS 2015).<br>
<br>
Thanks,<br>
Hans<br>
</blockquote></div>
</blockquote></div></blockquote></div>