<div dir="ltr"><div><div>Hello,<br><br></div>I believe I got the manifest and project converted to the new format required to install cleanly on VS2017. I'm wrapping things to send a patch, but I'm unsure about two things.<br><br></div><div>The new VSIX 3 manifest is only supported by Visual Studio 2012 or later, so just converting things would cause the extension to be unsupported by Visual Studio 2010 and below. As far as I could find out, the alternative would be mantaining a separate project for older versions (or just for VS2017), which adds a little complexity to the build process. Would the desire to support 2010 warrant that or is it old enough to disconsidered?<br><br></div><div>Also, could the solution and csproj be upgraded to VS15? This question may not apply if we decide to go with having two projects.<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 30, 2017 at 6:15 AM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks for doing this!<br>
<br>
As Paul said in his reply, either just sending the patch to<br>
cfe-commits, or uploading it to <a href="http://reviews.llvm.org" rel="noreferrer" target="_blank">reviews.llvm.org</a> works. In any case,<br>
please make sure I'm cc'd.<br>
<br>
If upgrading to the new manifest version turns out to be a lot of<br>
work, maybe we should commit your first patch in the meantime so the<br>
plugin can be used even if it displays a warning. Let me know how<br>
things develop.<br>
<br>
Cheers,<br>
Hans<br>
<div class="HOEnZb"><div class="h5"><br>
On Tue, Mar 28, 2017 at 5:55 AM, Hugo Puhlmann <<a href="mailto:hugopuhlmann@gmail.com">hugopuhlmann@gmail.com</a>> wrote:<br>
> Hi,<br>
><br>
> Visual Studio 2017 introduced a new version for its extension manifest, so<br>
> further work will be necessary for a "correct" migration. Right now the<br>
> extension works, but it triggers a warning message saying the extension is<br>
> not fully compatible with VS15.<br>
><br>
> I'll make the necessary changes and send an updated patch, so you can<br>
> disregard the first one.<br>
><br>
> In case anyone is curious, this post details the changes in the extension<br>
> SDK<br>
> <a href="http://www.visualstudioextensibility.com/2016/11/16/visual-studio-2017-rc-announced-extensions-need-some-changes/" rel="noreferrer" target="_blank">http://www.<wbr>visualstudioextensibility.com/<wbr>2016/11/16/visual-studio-2017-<wbr>rc-announced-extensions-need-<wbr>some-changes/</a><br>
><br>
> On Mon, Mar 27, 2017 at 10:57 PM, Hugo Puhlmann <<a href="mailto:hugopuhlmann@gmail.com">hugopuhlmann@gmail.com</a>><br>
> wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> Since there seems to be interest in VS15 support I'm submiting the patch I<br>
>> mentioned back in January.<br>
>><br>
>> I'm not sure if I can just submit it here, or if I should start another<br>
>> email thread for the patch itself, so apologies if it shouldn't have been<br>
>> here.<br>
>><br>
>> On Thu, Jan 26, 2017 at 8:39 PM, Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</a>> wrote:<br>
>>><br>
>>> On Sun, Jan 15, 2017 at 4:26 PM, Hugo Puhlmann via cfe-dev<br>
>>> <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>> wrote:<br>
>>> > Hi Piortr,<br>
>>> ><br>
>>> > I believe the one you linked is only supported by VS Code, not Visual<br>
>>> > Studio<br>
>>> > Community and similar.<br>
>>> ><br>
>>> > There's already a version for Visual Studio 2015 on<br>
>>> > <a href="http://llvm.org/builds" rel="noreferrer" target="_blank">http://llvm.org/builds</a>,<br>
>>> > but hat to build it myself for 2017RC.<br>
>>><br>
>>> That one is also available here:<br>
>>><br>
>>> <a href="https://marketplace.visualstudio.com/items?itemName=HansWennborg.ClangFormat" rel="noreferrer" target="_blank">https://marketplace.<wbr>visualstudio.com/items?<wbr>itemName=HansWennborg.<wbr>ClangFormat</a><br>
>>><br>
>>> If you have a patch for the manifest that makes the plugin work in<br>
>>> 2017RC, feel free to send it to the list. I don't think we'll want to<br>
>>> migrate the csproject yet though, as it needs to keep building with<br>
>>> 2015.<br>
>>><br>
>>> Cheers,<br>
>>> Hans<br>
>>><br>
>>> ><br>
>>> > On Sun, Jan 15, 2017 at 5:51 PM, Piotr Padlewski<br>
>>> > <<a href="mailto:piotr.padlewski@gmail.com">piotr.padlewski@gmail.com</a>><br>
>>> > wrote:<br>
>>> >><br>
>>> >> Hi,<br>
>>> >> I thought that there is already clang-format plugin<br>
>>> >> <a href="https://marketplace.visualstudio.com/items?itemName=xaver.clang-format" rel="noreferrer" target="_blank">https://marketplace.<wbr>visualstudio.com/items?<wbr>itemName=xaver.clang-format</a><br>
>>> >><br>
>>> >> Have you built your own plugin, or updated this one to make it work<br>
>>> >> with<br>
>>> >> new Visual?<br>
>>> >><br>
>>> >> Piotr<br>
>>> >><br>
>>> >> 2017-01-14 15:37 GMT+01:00 Hugo Puhlmann via cfe-dev<br>
>>> >> <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>>:<br>
>>> >>><br>
>>> >>> Hi,<br>
>>> >>><br>
>>> >>> I've recently managed to build and use a version of the clang-format<br>
>>> >>> plugin for VS 15.0, so I was wondering anyone would be interested in<br>
>>> >>> making<br>
>>> >>> the vsix package available somewhere public, like<br>
>>> >>> <a href="http://llvm.org/builds" rel="noreferrer" target="_blank">http://llvm.org/builds</a> or<br>
>>> >>> similar.<br>
>>> >>><br>
>>> >>> I could also submit a patch with the changes I've made, but since VS<br>
>>> >>> 15.0<br>
>>> >>> is still an RC and required me to upgrade the csproj version, it<br>
>>> >>> might be<br>
>>> >>> too early for that. The only change on the plugin itself was adding<br>
>>> >>> VS15 to<br>
>>> >>> the list of supported targets(<a href="http://source.extension.vsixmanifest.in" rel="noreferrer" target="_blank">source.extension.<wbr>vsixmanifest.in</a>), but<br>
>>> >>> I also<br>
>>> >>> needed to make some changes to be able to build the clang-format<br>
>>> >>> tool.<br>
>>> >>><br>
>>> >>> If anyone is interested, please let me know so I can send the<br>
>>> >>> package.<br>
>>> >>> --<br>
>>> >>> []'s,<br>
>>> >>> Hugo Puhlmann<br>
>>> >>><br>
>>> >>> ______________________________<wbr>_________________<br>
>>> >>> cfe-dev mailing list<br>
>>> >>> <a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a><br>
>>> >>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-dev</a><br>
>>> >>><br>
>>> >><br>
>>> ><br>
>>> ><br>
>>> ><br>
>>> > --<br>
>>> > []'s,<br>
>>> > Hugo Puhlmann<br>
>>> ><br>
>>> > ______________________________<wbr>_________________<br>
>>> > cfe-dev mailing list<br>
>>> > <a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a><br>
>>> > <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-dev</a><br>
>>> ><br>
>><br>
>><br>
>><br>
>><br>
>> --<br>
>> []'s,<br>
>> Hugo Puhlmann<br>
><br>
><br>
><br>
><br>
> --<br>
> []'s,<br>
> Hugo Puhlmann<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><font color="#999999" face="verdana, sans-serif">[]'s,</font><div><font color="#999999" face="verdana, sans-serif">Hugo Puhlmann</font></div></div>
</div>