Sorry I haven't had a chance to get back to this. Things got busy at work. I do plan to get back to this as I'm hoping to add some features to this extension :)<br><div class="gmail_quote"><div dir="ltr">On Thu, Sep 15, 2016 at 7:31 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"><div dir="ltr">Strange.  FWIW you can dump all the variables that are present in your environment.  You need to go to Tools -> Options -> Projects and Solutions -> Build and Run and choose either Normal, Detailed, or Diagnostic for the MSBuild project build output verbosity.  Then in the output window you will get a ton of spam, some of which is the set of all MSBuild variables you can take advantage of.</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Sep 15, 2016 at 4:25 PM 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">When I first opened the solution in VS it prompted me to install it and I did.<br>
<br>
On Thu, Sep 15, 2016 at 4:17 PM, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:<br>
> You may need to install the Visual Studio SDK.  Did you do that when you<br>
> initially installed VS 2015?<br>
><br>
> On Thu, Sep 15, 2016 at 4:15 PM Hans Wennborg <<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>> wrote:<br>
>><br>
>> Well, on my machine $(SDKToolsDir) doesn't work :-( I suspect the file<br>
>> will need manual tweaking by whoever is trying to build the plugin.<br>
>><br>
>> Anyway, I've updated the solution to build with VS2015 in r281648 and<br>
>> confirmed that it can still be used with older VS versions too.<br>
>><br>
>> Cheers,<br>
>> Hans<br>
>><br>
>> On Thu, Aug 18, 2016 at 7:11 PM, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>><br>
>> wrote:<br>
>> > The key.snk is generated when you build, the problem is the csproj file<br>
>> > hardcodes the directory to the sdk instead of using the appropriate<br>
>> > project<br>
>> > system variable like $(SDKToolsDir)<br>
>> ><br>
>> > On Thu, Aug 18, 2016 at 7:09 PM Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>><br>
>> > wrote:<br>
>> >><br>
>> >> Llvm doesn't support vs2012 anymore, as long as it supports vs2013 it's<br>
>> >> fine<br>
>> >> On Thu, Aug 18, 2016 at 7:07 PM Antonio Maiorano <<a href="mailto:amaiorano@gmail.com" target="_blank">amaiorano@gmail.com</a>><br>
>> >> wrote:<br>
>> >>><br>
>> >>> Hi,<br>
>> >>><br>
>> >>> What I meant by upgrade was simply making it build in VS 2015.<br>
>> >>> However,<br>
>> >>> you bring up a valid point about making sure the extension will<br>
>> >>> continue to<br>
>> >>> work in VS 2012. I will look into that. Like those references that go<br>
>> >>> from<br>
>> >>> 10 to 14 that point out; I wonder if instead I should be able to bring<br>
>> >>> in<br>
>> >>> those version 10 assemblies via NuGet. I'll take a closer look.<br>
>> >>><br>
>> >>> Part of my change, however, seems to imply that the extension (vsix)<br>
>> >>> project would not build correctly even in VS 2012. For instance, the<br>
>> >>> missing<br>
>> >>> Key.snk file. I don't have VS 2012 installed at the moment, so I<br>
>> >>> cannot<br>
>> >>> validate.<br>
>> >>><br>
>> >>> Thanks,<br>
>> >>><br>
>> >>> Antonio<br>
>> >>><br>
>> >>><br>
>> >>><br>
>> >>> On Thu, 18 Aug 2016 at 19:38 Hans Wennborg <<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>> wrote:<br>
>> >>>><br>
>> >>>> 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<br>
>> >>>> > 4.5, and<br>
>> >>>> > upgrading Microsoft.VisualStudio references to v14 versions<br>
>> >>>> > - Fix build by removing dependency on "Key.snk" file which was<br>
>> >>>> > never<br>
>> >>>> > checked<br>
>> >>>> > in (and not really required anyway)<br>
>> >>>> > - Add ".vs" directory to svn ignore (new folder that VS 2015<br>
>> >>>> > creates<br>
>> >>>> > 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<br>
>> >>>> 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>