<div dir="ltr">Updated version of patch attached with the changes you suggested. Let me know if there's anything else you'd like.<br><br><div class="gmail_quote"><div dir="ltr">On Tue, 29 Nov 2016 at 15:38 Hans Wennborg <<a href="mailto:hans@chromium.org">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">On Tue, Nov 29, 2016 at 12:01 PM, Antonio Maiorano <<a href="mailto:amaiorano@gmail.com" class="gmail_msg" target="_blank">amaiorano@gmail.com</a>> wrote:<br class="gmail_msg">
> On Tue, 29 Nov 2016 at 13:42 Hans Wennborg <<a href="mailto:hans@chromium.org" class="gmail_msg" target="_blank">hans@chromium.org</a>> wrote:<br class="gmail_msg">
>><br class="gmail_msg">
>> Very nice! I've tried this out and confirmed that the built plugin<br class="gmail_msg">
>> also works with older Visual Studio versions.<br class="gmail_msg">
>><br class="gmail_msg">
>> Some comments below:<br class="gmail_msg">
>><br class="gmail_msg">
>> > --- /dev/null<br class="gmail_msg">
>> > +++ b/tools/clang-format-vs/.gitignore<br class="gmail_msg">
>> > @@ -0,0 +1,11 @@<br class="gmail_msg">
>> > +# Visual Studio files<br class="gmail_msg">
>> > +.vs/<br class="gmail_msg">
>> > +/packages/<br class="gmail_msg">
>> > +/ClangFormat/obj/<br class="gmail_msg">
>> > +/ClangFormat/bin/<br class="gmail_msg">
>><br class="gmail_msg">
>> Should there really be a leading slash in these paths? Same below.<br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
> Without the leading slash, then it would match "ClangFormat/obj" anywhere in<br class="gmail_msg">
> the tree, rather than explicitly the one and only folder from the root. In<br class="gmail_msg">
> this case, it wouldn't matter much. If you prefer, I could just ignore<br class="gmail_msg">
> "obj/" and "bin/", etc, although I'd rather be explicit for Key.snk,<br class="gmail_msg">
> license.txt, etc.<br class="gmail_msg">
<br class="gmail_msg">
Interesting. I was just comparing with LLVM's root .gitignore, which<br class="gmail_msg">
doesn't have leading slashes for many dirs. Checking the gitignore<br class="gmail_msg">
doc, it sounds like you've got it right :-)<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
>> > +# Generated and copied files<br class="gmail_msg">
>> > +/ClangFormat/Key.snk<br class="gmail_msg">
>> > +/ClangFormat/license.txt<br class="gmail_msg">
>> > +/ClangFormat/clang-format.exe<br class="gmail_msg">
>> > +/ClangFormat/source.extension.vsixmanifest<br class="gmail_msg">
>> > diff --git a/tools/clang-format-vs/CMakeLists.txt<br class="gmail_msg">
>> > b/tools/clang-format-vs/CMakeLists.txt<br class="gmail_msg">
>> > index fd0d6b0..90f89d8 100644<br class="gmail_msg">
>> > --- a/tools/clang-format-vs/CMakeLists.txt<br class="gmail_msg">
>> > +++ b/tools/clang-format-vs/CMakeLists.txt<br class="gmail_msg">
>> > @@ -18,8 +18,13 @@ if (BUILD_CLANG_FORMAT_VS_PLUGIN)<br class="gmail_msg">
>> >    configure_file("<a href="http://source.extension.vsixmanifest.in" rel="noreferrer" class="gmail_msg" target="_blank">source.extension.vsixmanifest.in</a>"<br class="gmail_msg">
>> ><br class="gmail_msg">
>> > "${CMAKE_CURRENT_SOURCE_DIR}/ClangFormat/source.extension.vsixmanifest")<br class="gmail_msg">
>> ><br class="gmail_msg">
>> > -  add_custom_target(clang_format_vsix ALL<br class="gmail_msg">
>> > -      devenv "${CMAKE_CURRENT_SOURCE_DIR}/ClangFormat.sln" /Build<br class="gmail_msg">
>> > Release<br class="gmail_msg">
>> > +  find_program(NUGET_EXE nuget PATHS ${NUGET_EXE_PATH})<br class="gmail_msg">
>> > +  if (NOT NUGET_EXE)<br class="gmail_msg">
>> > + message(FATAL_ERROR "Could not find nuget.exe. Download from<br class="gmail_msg">
>> > <a href="https://www.nuget.org/nuget.exe" rel="noreferrer" class="gmail_msg" target="_blank">https://www.nuget.org/nuget.exe</a> and add to PATH or pass path via<br class="gmail_msg">
>> > NUGET_EXE_PATH var")<br class="gmail_msg">
>><br class="gmail_msg">
>> Can you break this string over multiple lines? I know there are pretty<br class="gmail_msg">
>> long lines in this file already, but it would be good to not make it<br class="gmail_msg">
>> worse.<br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
> Sure thing, I'll see what I can do :)<br class="gmail_msg">
><br class="gmail_msg">
>><br class="gmail_msg">
>><br class="gmail_msg">
>> Maybe it could be clarified that NUGET_EXE_PATH is the path to a<br class="gmail_msg">
>> directory containing nuget.exe, not to the file itself (I ran into<br class="gmail_msg">
>> this). Maybe NUGET_EXE_DIR would be a better name?<br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
> That would be a better name, will make that change.<br class="gmail_msg">
><br class="gmail_msg">
>><br class="gmail_msg">
>><br class="gmail_msg">
>> > +  endif()<br class="gmail_msg">
>> > +<br class="gmail_msg">
>> > +  add_custom_target(clang_format_vsix ALL<br class="gmail_msg">
>> > +  ${NUGET_EXE} restore "${CMAKE_CURRENT_SOURCE_DIR}/ClangFormat.sln" &<br class="gmail_msg">
>> > devenv "${CMAKE_CURRENT_SOURCE_DIR}/ClangFormat.sln" /Build Release<br class="gmail_msg">
>><br class="gmail_msg">
>> Is '&' a CMake thing, or should that be '&&'? Also, any chance of<br class="gmail_msg">
>> breaking these long lines?<br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
> '&' is a cmd.exe thing, but I can use '&&' - in fact, the latter only runs<br class="gmail_msg">
> the 2nd command if the first succeeds, while using '&' always runs both.<br class="gmail_msg">
> I'll make the change, and will try to shorten this line as well.<br class="gmail_msg">
><br class="gmail_msg">
>><br class="gmail_msg">
>><br class="gmail_msg">
>> >        DEPENDS clang_format_exe_for_vsix<br class="gmail_msg">
>> > "${CMAKE_CURRENT_SOURCE_DIR}/ClangFormat/source.extension.vsixmanifest"<br class="gmail_msg">
>> >        COMMAND ${CMAKE_COMMAND} -E copy_if_different<br class="gmail_msg">
>> ><br class="gmail_msg">
>> > "${CMAKE_CURRENT_SOURCE_DIR}/ClangFormat/bin/Release/ClangFormat.vsix"<br class="gmail_msg">
>> > diff --git a/tools/clang-format-vs/README.txt<br class="gmail_msg">
>> > b/tools/clang-format-vs/README.txt<br class="gmail_msg">
>> > index b23355d..44a071b 100644<br class="gmail_msg">
>> > --- a/tools/clang-format-vs/README.txt<br class="gmail_msg">
>> > +++ b/tools/clang-format-vs/README.txt<br class="gmail_msg">
>> > @@ -2,13 +2,23 @@ This directory contains a VSPackage project to<br class="gmail_msg">
>> > generate a Visual Studio extensio<br class="gmail_msg">
>> >  for clang-format.<br class="gmail_msg">
>> ><br class="gmail_msg">
>> >  Build prerequisites are:<br class="gmail_msg">
>> > -- Visual Studio 2013 Professional<br class="gmail_msg">
>> > -- Visual Studio 2013 SDK<br class="gmail_msg">
>> > -- Visual Studio 2010 Professional (?)<br class="gmail_msg">
>> > -- Visual Studio 2010 SDK (?)<br class="gmail_msg">
>> > +- Visual Studio 2015<br class="gmail_msg">
>> > +- Extensions SDK (you'll be prompted to install it if you open<br class="gmail_msg">
>> > ClangFormat.sln)<br class="gmail_msg">
>><br class="gmail_msg">
>> A very nice simplification :-)<br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
> Thanks :)<br class="gmail_msg">
><br class="gmail_msg">
>><br class="gmail_msg">
>><br class="gmail_msg">
>> ><br class="gmail_msg">
>> > -The extension is built using CMake by setting<br class="gmail_msg">
>> > BUILD_CLANG_FORMAT_VS_PLUGIN=ON<br class="gmail_msg">
>> > -when configuring a Clang build, and building the clang_format_vsix<br class="gmail_msg">
>> > target.<br class="gmail_msg">
>> > +The extension is built using CMake to generate the usual LLLVM.sln by<br class="gmail_msg">
>> > setting<br class="gmail_msg">
>><br class="gmail_msg">
>> An L too much in LLLVM.sln?<br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
> You know, that extra L in 'L'egendary LLVM? ;) Will remove!<br class="gmail_msg">
><br class="gmail_msg">
> Will update patch soon!<br class="gmail_msg">
><br class="gmail_msg">
>><br class="gmail_msg">
>><br class="gmail_msg">
>> On Mon, Nov 28, 2016 at 8:00 PM, Antonio Maiorano <<a href="mailto:amaiorano@gmail.com" class="gmail_msg" target="_blank">amaiorano@gmail.com</a>><br class="gmail_msg">
>> wrote:<br class="gmail_msg">
>> > I've attached a patch that works as discussed. When running CMake with<br class="gmail_msg">
>> > -DBUILD_CLANG_FORMAT_VS_PLUGIN=ON, it will look for nuget.exe in PATH,<br class="gmail_msg">
>> > or<br class="gmail_msg">
>> > you can pass in DNUGET_EXE_PATH=C:\nuget, for e.g.<br class="gmail_msg">
>> ><br class="gmail_msg">
>> ><br class="gmail_msg">
>> > On Mon, 28 Nov 2016 at 14:31 Antonio Maiorano <<a href="mailto:amaiorano@gmail.com" class="gmail_msg" target="_blank">amaiorano@gmail.com</a>><br class="gmail_msg">
>> > wrote:<br class="gmail_msg">
>> >><br class="gmail_msg">
>> >> Great, I'll get this working soon and attach a new patch :)<br class="gmail_msg">
>> >><br class="gmail_msg">
>> >> On Mon, 28 Nov 2016 at 14:27 Hans Wennborg <<a href="mailto:hans@chromium.org" class="gmail_msg" target="_blank">hans@chromium.org</a>> wrote:<br class="gmail_msg">
>> >>><br class="gmail_msg">
>> >>> On Mon, Nov 28, 2016 at 11:11 AM, Antonio Maiorano<br class="gmail_msg">
>> >>> <<a href="mailto:amaiorano@gmail.com" class="gmail_msg" target="_blank">amaiorano@gmail.com</a>><br class="gmail_msg">
>> >>> wrote:<br class="gmail_msg">
>> >>> >> It's built with the script in utils/release/build_llvm_package.bat<br class="gmail_msg">
>> >>> > which I run manually on my machine once every few weeks.<br class="gmail_msg">
>> >>> ><br class="gmail_msg">
>> >>> > Okay, that's good news. So the simplest path to success would be to<br class="gmail_msg">
>> >>> > require<br class="gmail_msg">
>> >>> > the user to either pass the path to CMake via an arg like<br class="gmail_msg">
>> >>> > -DNUGET_EXE_PATH,<br class="gmail_msg">
>> >>> > or if it's not defined, to assume it's already in PATH. This is the<br class="gmail_msg">
>> >>> > most<br class="gmail_msg">
>> >>> > future-proof solution as it will work with future versions of VS<br class="gmail_msg">
>> >>> > (2017<br class="gmail_msg">
>> >>> > RC<br class="gmail_msg">
>> >>> > just came out).<br class="gmail_msg">
>> >>> ><br class="gmail_msg">
>> >>> > I can still look into whether a vsix built with VS 2015 references<br class="gmail_msg">
>> >>> > will<br class="gmail_msg">
>> >>> > continue to work in older versions of VS, but even if this works, I<br class="gmail_msg">
>> >>> > feel<br class="gmail_msg">
>> >>> > like it's a temporary solution at best. There are other advantages<br class="gmail_msg">
>> >>> > to<br class="gmail_msg">
>> >>> > using<br class="gmail_msg">
>> >>> > NuGet here: it would allow us to more easily pin/upgrade which<br class="gmail_msg">
>> >>> > assemblies we<br class="gmail_msg">
>> >>> > want to use over time.<br class="gmail_msg">
>> >>> ><br class="gmail_msg">
>> >>> > If you're okay with it, I'll make the changes necessary to use<br class="gmail_msg">
>> >>> > -DNUGET_EXE_PATH, if defined, otherwise assume it's on PATH. This<br class="gmail_msg">
>> >>> > should be<br class="gmail_msg">
>> >>> > a simple change at this point.<br class="gmail_msg">
>> >>><br class="gmail_msg">
>> >>> That sounds good to me. There are already a bunch of prerequisites for<br class="gmail_msg">
>> >>> building the plugin, so adding this one doesn't seem unreasonable.<br class="gmail_msg">
>> >>> Especially since it seems it will simplify things to the point that<br class="gmail_msg">
>> >>> they might even work elsewhere than my own machine :-)<br class="gmail_msg">
>> >>><br class="gmail_msg">
>> >>><br class="gmail_msg">
>> >>> > On Mon, 28 Nov 2016 at 13:59 Hans Wennborg <<a href="mailto:hans@chromium.org" class="gmail_msg" target="_blank">hans@chromium.org</a>><br class="gmail_msg">
>> >>> > wrote:<br class="gmail_msg">
>> >>> >><br class="gmail_msg">
>> >>> >> On Mon, Nov 28, 2016 at 10:46 AM, Antonio Maiorano<br class="gmail_msg">
>> >>> >> <<a href="mailto:amaiorano@gmail.com" class="gmail_msg" target="_blank">amaiorano@gmail.com</a>><br class="gmail_msg">
>> >>> >> wrote:<br class="gmail_msg">
>> >>> >> > Okay, I'll see if upgrading to the 2015 assemblies would allow<br class="gmail_msg">
>> >>> >> > the<br class="gmail_msg">
>> >>> >> > VSIX<br class="gmail_msg">
>> >>> >> > to<br class="gmail_msg">
>> >>> >> > keep working in older versions of VS.<br class="gmail_msg">
>> >>> >> ><br class="gmail_msg">
>> >>> >> > Still waiting on an answer to this question:<br class="gmail_msg">
>> >>> >> ><br class="gmail_msg">
>> >>> >> >> In either case, though, I must ask: how is the offical vsix<br class="gmail_msg">
>> >>> >> >> that's<br class="gmail_msg">
>> >>> >> >> available on <a href="http://llvm.org/builds/" rel="noreferrer" class="gmail_msg" target="_blank">http://llvm.org/builds/</a> get built? Is it part of an<br class="gmail_msg">
>> >>> >> >> automated<br class="gmail_msg">
>> >>> >> >> Clang build, or is it built and uploaded manually? If it's<br class="gmail_msg">
>> >>> >> >> automated,<br class="gmail_msg">
>> >>> >> >> then<br class="gmail_msg">
>> >>> >> >> having to download and point to nuget.exe won't work.<br class="gmail_msg">
>> >>> >><br class="gmail_msg">
>> >>> >> It's built with the script in utils/release/build_llvm_package.bat<br class="gmail_msg">
>> >>> >> which I run manually on my machine once every few weeks.<br class="gmail_msg">
>> >>> >><br class="gmail_msg">
>> >>> >><br class="gmail_msg">
>> >>> >> > On Mon, 28 Nov 2016 at 13:04 Hans Wennborg <<a href="mailto:hans@chromium.org" class="gmail_msg" target="_blank">hans@chromium.org</a>><br class="gmail_msg">
>> >>> >> > wrote:<br class="gmail_msg">
>> >>> >> >><br class="gmail_msg">
>> >>> >> >> On Fri, Nov 25, 2016 at 6:58 PM, Antonio Maiorano<br class="gmail_msg">
>> >>> >> >> <<a href="mailto:amaiorano@gmail.com" class="gmail_msg" target="_blank">amaiorano@gmail.com</a>><br class="gmail_msg">
>> >>> >> >> wrote:<br class="gmail_msg">
>> >>> >> >> > Ah, no, that's not what I meant. The required referenced<br class="gmail_msg">
>> >>> >> >> > assemblies<br class="gmail_msg">
>> >>> >> >> > are<br class="gmail_msg">
>> >>> >> >> > versions that are normally installed with VS 2010.<br class="gmail_msg">
>> >>> >> >> ><br class="gmail_msg">
>> >>> >> >> > The first time I worked on this, I had upgraded the referenced<br class="gmail_msg">
>> >>> >> >> > assemblies to<br class="gmail_msg">
>> >>> >> >> > the ones that ship with VS 2015, but then there was question<br class="gmail_msg">
>> >>> >> >> > of<br class="gmail_msg">
>> >>> >> >> > whether<br class="gmail_msg">
>> >>> >> >> > or<br class="gmail_msg">
>> >>> >> >> > not the VSIX would continue to work with VS 2010/2012/2013.<br class="gmail_msg">
>> >>> >> >> > I'm<br class="gmail_msg">
>> >>> >> >> > not<br class="gmail_msg">
>> >>> >> >> > sure<br class="gmail_msg">
>> >>> >> >> > if<br class="gmail_msg">
>> >>> >> >> > it would work, but I guess I can try to figure that out.<br class="gmail_msg">
>> >>> >> >><br class="gmail_msg">
>> >>> >> >> Let me know if you figure this one out. It sounds like it would<br class="gmail_msg">
>> >>> >> >> simplify things a lot.<br class="gmail_msg">
>> >>> >> >><br class="gmail_msg">
>> >>> >> >> > In any case, what I discovered is that the "right" way to do<br class="gmail_msg">
>> >>> >> >> > things<br class="gmail_msg">
>> >>> >> >> > to<br class="gmail_msg">
>> >>> >> >> > make<br class="gmail_msg">
>> >>> >> >> > sure your extension compiles in future versions of VS is to<br class="gmail_msg">
>> >>> >> >> > use<br class="gmail_msg">
>> >>> >> >> > NuGet<br class="gmail_msg">
>> >>> >> >> > to<br class="gmail_msg">
>> >>> >> >> > automatically pull in the required assemblies, or to check<br class="gmail_msg">
>> >>> >> >> > them<br class="gmail_msg">
>> >>> >> >> > in<br class="gmail_msg">
>> >>> >> >> > and<br class="gmail_msg">
>> >>> >> >> > reference them directly. The former would be better except for<br class="gmail_msg">
>> >>> >> >> > the<br class="gmail_msg">
>> >>> >> >> > problem<br class="gmail_msg">
>> >>> >> >> > of CLI builds as I described in my earlier email.<br class="gmail_msg">
>> >>> >> >> ><br class="gmail_msg">
>> >>> >> >> ><br class="gmail_msg">
>> >>> >> >> ><br class="gmail_msg">
>> >>> >> >> > On Fri, 25 Nov 2016 at 21:47 Zachary Turner<br class="gmail_msg">
>> >>> >> >> > <<a href="mailto:zturner@google.com" class="gmail_msg" target="_blank">zturner@google.com</a>><br class="gmail_msg">
>> >>> >> >> > wrote:<br class="gmail_msg">
>> >>> >> >> >><br class="gmail_msg">
>> >>> >> >> >> Sorry, i think I misunderstood the original option 1. I<br class="gmail_msg">
>> >>> >> >> >> interpreted<br class="gmail_msg">
>> >>> >> >> >> it<br class="gmail_msg">
>> >>> >> >> >> as<br class="gmail_msg">
>> >>> >> >> >> just committing changes to the vsix manifest to reference a<br class="gmail_msg">
>> >>> >> >> >> specific<br class="gmail_msg">
>> >>> >> >> >> version<br class="gmail_msg">
>> >>> >> >> >> of the assembly which we assume to be present since it should<br class="gmail_msg">
>> >>> >> >> >> be<br class="gmail_msg">
>> >>> >> >> >> automatically installed with vs 2015. Is this not possible?<br class="gmail_msg">
>> >>> >> >> >> Can't we<br class="gmail_msg">
>> >>> >> >> >> just<br class="gmail_msg">
>> >>> >> >> >> point the manifest to the version installed with vs?<br class="gmail_msg">
>> >>> >> >> >> On Fri, Nov 25, 2016 at 6:20 PM Antonio Maiorano<br class="gmail_msg">
>> >>> >> >> >> <<a href="mailto:amaiorano@gmail.com" class="gmail_msg" target="_blank">amaiorano@gmail.com</a>><br class="gmail_msg">
>> >>> >> >> >> wrote:<br class="gmail_msg">
>> >>> >> >> >>><br class="gmail_msg">
>> >>> >> >> >>> Hi again,<br class="gmail_msg">
>> >>> >> >> >>><br class="gmail_msg">
>> >>> >> >> >>> I've made the changes so that the required assemblies are<br class="gmail_msg">
>> >>> >> >> >>> committed,<br class="gmail_msg">
>> >>> >> >> >>> so<br class="gmail_msg">
>> >>> >> >> >>> now we can build the clang-format-vsix with just VS 2015.<br class="gmail_msg">
>> >>> >> >> >>> Since<br class="gmail_msg">
>> >>> >> >> >>> the<br class="gmail_msg">
>> >>> >> >> >>> patch<br class="gmail_msg">
>> >>> >> >> >>> set is around 9 mb, I'm providing a link to it on my Dropbox<br class="gmail_msg">
>> >>> >> >> >>> (if<br class="gmail_msg">
>> >>> >> >> >>> you'd<br class="gmail_msg">
>> >>> >> >> >>> rather I attach it, let me know):<br class="gmail_msg">
>> >>> >> >> >>><br class="gmail_msg">
>> >>> >> >> >>><br class="gmail_msg">
>> >>> >> >> >>><br class="gmail_msg">
>> >>> >> >> >>><br class="gmail_msg">
>> >>> >> >> >>><br class="gmail_msg">
>> >>> >> >> >>><br class="gmail_msg">
>> >>> >> >> >>> <a href="https://dl.dropboxusercontent.com/u/10504225/llvm-patches/0001-Fix-VS2015-build-of-clang-format-vsix-by-committing-.patch" rel="noreferrer" class="gmail_msg" target="_blank">https://dl.dropboxusercontent.com/u/10504225/llvm-patches/0001-Fix-VS2015-build-of-clang-format-vsix-by-committing-.patch</a><br class="gmail_msg">
>> >>> >> >> >>><br class="gmail_msg">
>> >>> >> >> >>> Note that it's a git patch set, for which I followed the<br class="gmail_msg">
>> >>> >> >> >>> instructions<br class="gmail_msg">
>> >>> >> >> >>> here.<br class="gmail_msg">
>> >>> >> >> >>><br class="gmail_msg">
>> >>> >> >> >>> Thanks.<br class="gmail_msg">
>> >>> >> >> >>><br class="gmail_msg">
>> >>> >> >> >>> On Thu, 24 Nov 2016 at 15:45 Antonio Maiorano<br class="gmail_msg">
>> >>> >> >> >>> <<a href="mailto:amaiorano@gmail.com" class="gmail_msg" target="_blank">amaiorano@gmail.com</a>><br class="gmail_msg">
>> >>> >> >> >>> wrote:<br class="gmail_msg">
>> >>> >> >> >>>><br class="gmail_msg">
>> >>> >> >> >>>> Okay, that's fine, I'll go for that and if all looks good,<br class="gmail_msg">
>> >>> >> >> >>>> will<br class="gmail_msg">
>> >>> >> >> >>>> attach a<br class="gmail_msg">
>> >>> >> >> >>>> patch.<br class="gmail_msg">
>> >>> >> >> >>>><br class="gmail_msg">
>> >>> >> >> >>>> Thanks.<br class="gmail_msg">
>> >>> >> >> >>>><br class="gmail_msg">
>> >>> >> >> >>>> On Thu, 24 Nov 2016 at 15:09 Zachary Turner<br class="gmail_msg">
>> >>> >> >> >>>> <<a href="mailto:zturner@google.com" class="gmail_msg" target="_blank">zturner@google.com</a>><br class="gmail_msg">
>> >>> >> >> >>>> wrote:<br class="gmail_msg">
>> >>> >> >> >>>>><br class="gmail_msg">
>> >>> >> >> >>>>> I would use the first solution. We lock ourselves to<br class="gmail_msg">
>> >>> >> >> >>>>> specific<br class="gmail_msg">
>> >>> >> >> >>>>> versions<br class="gmail_msg">
>> >>> >> >> >>>>> of vs, so i think it's fine to do the same with the<br class="gmail_msg">
>> >>> >> >> >>>>> assemblies<br class="gmail_msg">
>> >>> >> >> >>>>> and<br class="gmail_msg">
>> >>> >> >> >>>>> deal with<br class="gmail_msg">
>> >>> >> >> >>>>> it only when we upgrade<br class="gmail_msg">
>> >>> >> >> >>>>> On Thu, Nov 24, 2016 at 11:29 AM Antonio Maiorano<br class="gmail_msg">
>> >>> >> >> >>>>> <<a href="mailto:amaiorano@gmail.com" class="gmail_msg" target="_blank">amaiorano@gmail.com</a>><br class="gmail_msg">
>> >>> >> >> >>>>> wrote:<br class="gmail_msg">
>> >>> >> >> >>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>> Hi Hans,<br class="gmail_msg">
>> >>> >> >> >>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>> I saw that on September 15th, you checked in a change:<br class="gmail_msg">
>> >>> >> >> >>>>>> clang-format<br class="gmail_msg">
>> >>> >> >> >>>>>> VS<br class="gmail_msg">
>> >>> >> >> >>>>>> plugin: upgrade the project files to VS2015.<br class="gmail_msg">
>> >>> >> >> >>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>> When I open the latest version of ClangFormat.sln on a<br class="gmail_msg">
>> >>> >> >> >>>>>> machine<br class="gmail_msg">
>> >>> >> >> >>>>>> that<br class="gmail_msg">
>> >>> >> >> >>>>>> has only VS 2015, it doesn't build. The reason is that<br class="gmail_msg">
>> >>> >> >> >>>>>> some<br class="gmail_msg">
>> >>> >> >> >>>>>> of<br class="gmail_msg">
>> >>> >> >> >>>>>> the<br class="gmail_msg">
>> >>> >> >> >>>>>> referenced assemblies are from VS 2010.<br class="gmail_msg">
>> >>> >> >> >>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>> <Reference Include="Microsoft.VisualStudio.CoreUtility,<br class="gmail_msg">
>> >>> >> >> >>>>>> Version=10.0.0.0, Culture=neutral,<br class="gmail_msg">
>> >>> >> >> >>>>>> PublicKeyToken=b03f5f7f11d50a3a,<br class="gmail_msg">
>> >>> >> >> >>>>>> processorArchitecture=MSIL" /> <Reference<br class="gmail_msg">
>> >>> >> >> >>>>>> Include="Microsoft.VisualStudio.Editor, Version=10.0.0.0,<br class="gmail_msg">
>> >>> >> >> >>>>>> Culture=neutral,<br class="gmail_msg">
>> >>> >> >> >>>>>> PublicKeyToken=b03f5f7f11d50a3a,<br class="gmail_msg">
>> >>> >> >> >>>>>> processorArchitecture=MSIL"<br class="gmail_msg">
>> >>> >> >> >>>>>> /><br class="gmail_msg">
>> >>> >> >> >>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>> What happens is that when building, these specific<br class="gmail_msg">
>> >>> >> >> >>>>>> assemblies<br class="gmail_msg">
>> >>> >> >> >>>>>> are<br class="gmail_msg">
>> >>> >> >> >>>>>> not<br class="gmail_msg">
>> >>> >> >> >>>>>> found. I suspect you have VS 2010 installed on your<br class="gmail_msg">
>> >>> >> >> >>>>>> machine,<br class="gmail_msg">
>> >>> >> >> >>>>>> which<br class="gmail_msg">
>> >>> >> >> >>>>>> is why<br class="gmail_msg">
>> >>> >> >> >>>>>> you don't get these build errors.<br class="gmail_msg">
>> >>> >> >> >>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>> The recommended way to handle this is to make use of<br class="gmail_msg">
>> >>> >> >> >>>>>> NuGet<br class="gmail_msg">
>> >>> >> >> >>>>>> to<br class="gmail_msg">
>> >>> >> >> >>>>>> have<br class="gmail_msg">
>> >>> >> >> >>>>>> it<br class="gmail_msg">
>> >>> >> >> >>>>>> automatically download the required assemblies. I have<br class="gmail_msg">
>> >>> >> >> >>>>>> made<br class="gmail_msg">
>> >>> >> >> >>>>>> the<br class="gmail_msg">
>> >>> >> >> >>>>>> changes<br class="gmail_msg">
>> >>> >> >> >>>>>> locally to get this working, and it works great when<br class="gmail_msg">
>> >>> >> >> >>>>>> building<br class="gmail_msg">
>> >>> >> >> >>>>>> ClangFormat.sln from within Visual Studio; however,<br class="gmail_msg">
>> >>> >> >> >>>>>> building<br class="gmail_msg">
>> >>> >> >> >>>>>> from<br class="gmail_msg">
>> >>> >> >> >>>>>> the CLI<br class="gmail_msg">
>> >>> >> >> >>>>>> doesn't work out of the box because you must explicitly<br class="gmail_msg">
>> >>> >> >> >>>>>> run<br class="gmail_msg">
>> >>> >> >> >>>>>> 'nuget.exe<br class="gmail_msg">
>> >>> >> >> >>>>>> restore ClangFormat.sln' before running msbuild (or<br class="gmail_msg">
>> >>> >> >> >>>>>> devenv.exe<br class="gmail_msg">
>> >>> >> >> >>>>>> in<br class="gmail_msg">
>> >>> >> >> >>>>>> our case).<br class="gmail_msg">
>> >>> >> >> >>>>>> The problem is that nuget.exe isn't something that's<br class="gmail_msg">
>> >>> >> >> >>>>>> easily<br class="gmail_msg">
>> >>> >> >> >>>>>> found/accessible<br class="gmail_msg">
>> >>> >> >> >>>>>> on Windows, even once installed as an extension in VS.<br class="gmail_msg">
>> >>> >> >> >>>>>> This<br class="gmail_msg">
>> >>> >> >> >>>>>> is a<br class="gmail_msg">
>> >>> >> >> >>>>>> known<br class="gmail_msg">
>> >>> >> >> >>>>>> problem and the current best solution requires<br class="gmail_msg">
>> >>> >> >> >>>>>> downloading<br class="gmail_msg">
>> >>> >> >> >>>>>> and<br class="gmail_msg">
>> >>> >> >> >>>>>> making<br class="gmail_msg">
>> >>> >> >> >>>>>> nuget.exe available in PATH.<br class="gmail_msg">
>> >>> >> >> >>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>> So now i'm faced with figuring out how best to solve this<br class="gmail_msg">
>> >>> >> >> >>>>>> so<br class="gmail_msg">
>> >>> >> >> >>>>>> that<br class="gmail_msg">
>> >>> >> >> >>>>>> the<br class="gmail_msg">
>> >>> >> >> >>>>>> custom build step in this CMakeLists.txt that runs devenv<br class="gmail_msg">
>> >>> >> >> >>>>>> doesn't<br class="gmail_msg">
>> >>> >> >> >>>>>> fail:<br class="gmail_msg">
>> >>> >> >> >>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>> devenv "${CMAKE_CURRENT_SOURCE_DIR}/ClangFormat.sln"<br class="gmail_msg">
>> >>> >> >> >>>>>> /Build<br class="gmail_msg">
>> >>> >> >> >>>>>> Release<br class="gmail_msg">
>> >>> >> >> >>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>> There are a few options:<br class="gmail_msg">
>> >>> >> >> >>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>> 1) Forget NuGet and just commit the referenced<br class="gmail_msg">
>> >>> >> >> >>>>>> assemblies.<br class="gmail_msg">
>> >>> >> >> >>>>>> This<br class="gmail_msg">
>> >>> >> >> >>>>>> is<br class="gmail_msg">
>> >>> >> >> >>>>>> the<br class="gmail_msg">
>> >>> >> >> >>>>>> simplest solution, but is more annoying to manage if we<br class="gmail_msg">
>> >>> >> >> >>>>>> want<br class="gmail_msg">
>> >>> >> >> >>>>>> to<br class="gmail_msg">
>> >>> >> >> >>>>>> upgrade the<br class="gmail_msg">
>> >>> >> >> >>>>>> versions of these assemblies later.<br class="gmail_msg">
>> >>> >> >> >>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>> 2) Commit nuget.exe to the repo and just use it. This is<br class="gmail_msg">
>> >>> >> >> >>>>>> simple<br class="gmail_msg">
>> >>> >> >> >>>>>> enough, but I'm not sure how people feel about committing<br class="gmail_msg">
>> >>> >> >> >>>>>> binaries,<br class="gmail_msg">
>> >>> >> >> >>>>>> and it<br class="gmail_msg">
>> >>> >> >> >>>>>> would be frozen at whatever version we commit.<br class="gmail_msg">
>> >>> >> >> >>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>> 3) Do some form of wget to grab the latest nuget.exe from<br class="gmail_msg">
>> >>> >> >> >>>>>> "<a href="https://nuget.org/nuget.exe" rel="noreferrer" class="gmail_msg" target="_blank">https://nuget.org/nuget.exe</a>" in CMake and invoke it. I'm<br class="gmail_msg">
>> >>> >> >> >>>>>> not<br class="gmail_msg">
>> >>> >> >> >>>>>> yet<br class="gmail_msg">
>> >>> >> >> >>>>>> sure<br class="gmail_msg">
>> >>> >> >> >>>>>> what's the simplest way to do this. Powershell could be<br class="gmail_msg">
>> >>> >> >> >>>>>> used,<br class="gmail_msg">
>> >>> >> >> >>>>>> but<br class="gmail_msg">
>> >>> >> >> >>>>>> there are<br class="gmail_msg">
>> >>> >> >> >>>>>> security annoyances to deal with.<br class="gmail_msg">
>> >>> >> >> >>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>> That's all I can come up with so far. Would love to hear<br class="gmail_msg">
>> >>> >> >> >>>>>> from<br class="gmail_msg">
>> >>> >> >> >>>>>> you<br class="gmail_msg">
>> >>> >> >> >>>>>> guys<br class="gmail_msg">
>> >>> >> >> >>>>>> if you have any ideas or opinions on this. If you want I<br class="gmail_msg">
>> >>> >> >> >>>>>> can<br class="gmail_msg">
>> >>> >> >> >>>>>> send<br class="gmail_msg">
>> >>> >> >> >>>>>> you a<br class="gmail_msg">
>> >>> >> >> >>>>>> patch of what I've got so far if that helps.<br class="gmail_msg">
>> >>> >> >> >>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>> Thanks,<br class="gmail_msg">
>> >>> >> >> >>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>> Antonio Maiorano<br class="gmail_msg">
>> >>> >> >> >>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>> On Thu, 15 Sep 2016 at 19:35 Antonio Maiorano<br class="gmail_msg">
>> >>> >> >> >>>>>> <<a href="mailto:amaiorano@gmail.com" class="gmail_msg" target="_blank">amaiorano@gmail.com</a>><br class="gmail_msg">
>> >>> >> >> >>>>>> wrote:<br class="gmail_msg">
>> >>> >> >> >>>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>>> Sorry I haven't had a chance to get back to this. Things<br class="gmail_msg">
>> >>> >> >> >>>>>>> got<br class="gmail_msg">
>> >>> >> >> >>>>>>> busy<br class="gmail_msg">
>> >>> >> >> >>>>>>> at<br class="gmail_msg">
>> >>> >> >> >>>>>>> work. I do plan to get back to this as I'm hoping to add<br class="gmail_msg">
>> >>> >> >> >>>>>>> some<br class="gmail_msg">
>> >>> >> >> >>>>>>> features to<br class="gmail_msg">
>> >>> >> >> >>>>>>> this extension :)<br class="gmail_msg">
>> >>> >> >> >>>>>>> On Thu, Sep 15, 2016 at 7:31 PM Zachary Turner<br class="gmail_msg">
>> >>> >> >> >>>>>>> <<a href="mailto:zturner@google.com" class="gmail_msg" target="_blank">zturner@google.com</a>><br class="gmail_msg">
>> >>> >> >> >>>>>>> wrote:<br class="gmail_msg">
>> >>> >> >> >>>>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>>>> Strange.  FWIW you can dump all the variables that are<br class="gmail_msg">
>> >>> >> >> >>>>>>>> present<br class="gmail_msg">
>> >>> >> >> >>>>>>>> in<br class="gmail_msg">
>> >>> >> >> >>>>>>>> your environment.  You need to go to Tools -> Options<br class="gmail_msg">
>> >>> >> >> >>>>>>>> -><br class="gmail_msg">
>> >>> >> >> >>>>>>>> Projects<br class="gmail_msg">
>> >>> >> >> >>>>>>>> and<br class="gmail_msg">
>> >>> >> >> >>>>>>>> Solutions -> Build and Run and choose either Normal,<br class="gmail_msg">
>> >>> >> >> >>>>>>>> Detailed,<br class="gmail_msg">
>> >>> >> >> >>>>>>>> or<br class="gmail_msg">
>> >>> >> >> >>>>>>>> Diagnostic<br class="gmail_msg">
>> >>> >> >> >>>>>>>> for the MSBuild project build output verbosity.  Then<br class="gmail_msg">
>> >>> >> >> >>>>>>>> in<br class="gmail_msg">
>> >>> >> >> >>>>>>>> the<br class="gmail_msg">
>> >>> >> >> >>>>>>>> output window<br class="gmail_msg">
>> >>> >> >> >>>>>>>> you will get a ton of spam, some of which is the set of<br class="gmail_msg">
>> >>> >> >> >>>>>>>> all<br class="gmail_msg">
>> >>> >> >> >>>>>>>> MSBuild<br class="gmail_msg">
>> >>> >> >> >>>>>>>> variables you can take advantage of.<br class="gmail_msg">
>> >>> >> >> >>>>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>>>> On Thu, Sep 15, 2016 at 4:25 PM Hans Wennborg<br class="gmail_msg">
>> >>> >> >> >>>>>>>> <<a href="mailto:hans@chromium.org" class="gmail_msg" target="_blank">hans@chromium.org</a>><br class="gmail_msg">
>> >>> >> >> >>>>>>>> wrote:<br class="gmail_msg">
>> >>> >> >> >>>>>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> When I first opened the solution in VS it prompted me<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> to<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> install<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> it<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> and I did.<br class="gmail_msg">
>> >>> >> >> >>>>>>>>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> On Thu, Sep 15, 2016 at 4:17 PM, Zachary Turner<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> <<a href="mailto:zturner@google.com" class="gmail_msg" target="_blank">zturner@google.com</a>> wrote:<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> > You may need to install the Visual Studio SDK.  Did<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> > you<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> > do<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> > that<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> > when you<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> > initially installed VS 2015?<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> ><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> > On Thu, Sep 15, 2016 at 4:15 PM Hans Wennborg<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> > <<a href="mailto:hans@chromium.org" class="gmail_msg" target="_blank">hans@chromium.org</a>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> > wrote:<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> Well, on my machine $(SDKToolsDir) doesn't work :-(<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> I<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> suspect<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> the file<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> will need manual tweaking by whoever is trying to<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> build<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> the<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> plugin.<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> Anyway, I've updated the solution to build with<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> VS2015<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> in<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> r281648 and<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> confirmed that it can still be used with older VS<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> versions<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> too.<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> Cheers,<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> Hans<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> On Thu, Aug 18, 2016 at 7:11 PM, Zachary Turner<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> <<a href="mailto:zturner@google.com" class="gmail_msg" target="_blank">zturner@google.com</a>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> wrote:<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> > The key.snk is generated when you build, the<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> > problem<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> > is<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> > the<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> > csproj file<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> > hardcodes the directory to the sdk instead of<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> > using<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> > the<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> > appropriate<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> > project<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> > system variable like $(SDKToolsDir)<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> ><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> > On Thu, Aug 18, 2016 at 7:09 PM Zachary Turner<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> > <<a href="mailto:zturner@google.com" class="gmail_msg" target="_blank">zturner@google.com</a>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> > wrote:<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >> Llvm doesn't support vs2012 anymore, as long as<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >> it<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >> supports<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >> vs2013 it's<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >> fine<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >> On Thu, Aug 18, 2016 at 7:07 PM Antonio Maiorano<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >> <<a href="mailto:amaiorano@gmail.com" class="gmail_msg" target="_blank">amaiorano@gmail.com</a>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >> wrote:<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> Hi,<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> What I meant by upgrade was simply making it<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> build<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> in<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> VS<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> 2015.<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> However,<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> you bring up a valid point about making sure<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> the<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> extension<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> will<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> continue to<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> work in VS 2012. I will look into that. Like<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> those<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> references that go<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> from<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> 10 to 14 that point out; I wonder if instead I<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> should<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> be<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> able to bring<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> in<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> those version 10 assemblies via NuGet. I'll<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> take a<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> closer<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> look.<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> Part of my change, however, seems to imply that<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> the<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> extension (vsix)<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> project would not build correctly even in VS<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> 2012.<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> For<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> instance, the<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> missing<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> Key.snk file. I don't have VS 2012 installed at<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> the<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> moment,<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> so I<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> cannot<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> validate.<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> Thanks,<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> Antonio<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> On Thu, 18 Aug 2016 at 19:38 Hans Wennborg<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>> <<a href="mailto:hans@chromium.org" class="gmail_msg" target="_blank">hans@chromium.org</a>> wrote:<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> Hi Antonio,<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> On Wed, Aug 17, 2016 at 8:15 AM, Antonio<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> Maiorano<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> via<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> cfe-commits<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> <<a href="mailto:cfe-commits@lists.llvm.org" class="gmail_msg" target="_blank">cfe-commits@lists.llvm.org</a>> wrote:<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > This patch for clang-format-vs includes the<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > following:<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> ><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > - Upgrade to VS 2015, including .NET<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > framework<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > upgrade<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > from 4.0 to<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > 4.5, and<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > upgrading Microsoft.VisualStudio references<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > to<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > v14<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > versions<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > - Fix build by removing dependency on<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > "Key.snk"<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > file<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > which was<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > never<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > checked<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > in (and not really required anyway)<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > - Add ".vs" directory to svn ignore (new<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > folder<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > that<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > VS<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > 2015<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > creates<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > for<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> > user settings)<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> "What does "Upgrade to VS 2015 mean? Adding<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> support<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> for<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> running the<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> plugin in VS2015, or does it mean requiring<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> VS2015<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> for<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> building?<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> +zturner: I thought the plugin already worked<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> in<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> VS<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> 2015?<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> I mostly just build the plugin without knowing<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> exactly<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> how<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> this stuff<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> works, but looking at the patch I'm worried<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> that<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> you're<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> increasing<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> the<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> required version for building it? I see a<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> bunch<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> of<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> values<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> going from<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> 10 (VS 2012) to 14 (VS 2015).<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>><br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> Thanks,<br class="gmail_msg">
>> >>> >> >> >>>>>>>>> >> >>>> Hans<br class="gmail_msg">
</blockquote></div></div>