<div><br></div><div><br><div class="gmail_quote"><div dir="ltr">On Mon, Aug 6, 2018 at 12:18 PM Hans Wennborg via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">hans added a comment.<br>
<br>
In <a href="https://reviews.llvm.org/D50335#1189459" rel="noreferrer" target="_blank">https://reviews.llvm.org/D50335#1189459</a>, @zturner wrote:<br>
<br>
> Btw, unfortunately i had a few more changes to the csproj file that I may<br>
>  have forgotten to upstream before i left. IIRC, they were changing the<br>
>  author name to LLVMExtensions , deleting the part in the csproj about<br>
>  Brutal Strong name signer, setting the key file in project settings, and<br>
>  fixing the path to sn.exe.<br>
<br>
<br>
Ouch, it's not so great that the published extension was built from source that's not checked in. But I guess publishing an updated version can wait until you're back.</blockquote><div dir="auto">It’s only a 3 or 4 line change, and you won’t be able to upload until they’re fixed so it shouldn’t be too hard if Youre willing to give it a try.</div><div dir="auto"><br></div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
In <a href="https://reviews.llvm.org/D50335#1189459" rel="noreferrer" target="_blank">https://reviews.llvm.org/D50335#1189459</a>, @zturner wrote:<br>
<br>
> Edit: just saw your update. That is such weird installer behavior. What<br>
>  goes in the LLVM key above that? Maybe the installer behavior changed and<br>
>  we need to support both?<br>
<br>
<br>
I tried an old installer (3.8.0) and it sets the same registry keys.<br>
<br>
I don't know if it's that weird. I think cmake/cpack just figures the package vendor is LLVM and the product is also called LLVM, and that's how it ended up there.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D50335" rel="noreferrer" target="_blank">https://reviews.llvm.org/D50335</a><br>
<br>
<br>
<br>
</blockquote></div></div>