<div dir="ltr">Hi,<div><br></div><div>sorry for the delay. I thought maybe Emil is on holiday. I plan to push this on Monday evening (EDT) if there is no response by then.</div><div><br></div><div>Jan</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, May 13, 2018 at 3:56 AM, Kai Wasserbäch <span dir="ltr"><<a href="mailto:kai@dev.carbon-project.org" target="_blank">kai@dev.carbon-project.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ping! Can somebody *please* commit this patch? It fixes an FTBFS, has two T-b<br>
and one A-b.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
Kai Wasserbäch wrote on 07.05.2018 16:48:<br>
> Jan Vesely wrote on 02.05.2018 22:38:<br>
>> On Wed, 2018-05-02 at 18:38 +0200, Kai Wasserbäch wrote:<br>
>>> [...]<br>
>><br>
>> Thank for looking into this. We probably need CLANG_LIBS handling<br>
>> similar to LLVM_LIBS. I agree this is the best fix for now.<br>
>><br>
>> Acked-by: Jan Vesely <<a href="mailto:jan.vesely@rutgers.edu">jan.vesely@rutgers.edu</a>><br>
>><br>
>> libclang.so might be a solkution, but I'm not sure how it interacts<br>
>> with older or static build clang. It's also weird that we are linking<br>
>> to clang here instead of clover which actually uses clang symbols.<br>
>><br>
>> @Emil, are you OK with this patch?<br>
> <br>
> Gentle ping.<br>
> <br>
>>>>>>>         -lclangDriver \<br>
>>>>>>>         -lclangSerialization \<br>
>>>>>>> -       -lclangCodeGen \<br>
>>>>>><br>
>>>>>> Is this change related?<br>
>>>>><br>
>>>>> Not really, just a minor clean-up while I was busy a few lines above.<br>
>>>>> "clangCodeGen" is already named on the first Clang library line.<br>
>>>><br>
>>>> ah, all right, maybe mention it in the commit message?<br>
>>><br>
>>> Do I need to resend the patch for that or can you just add a line like "This<br>
>>> change also removes the duplicate clangCodeGen line (trivial change)." before<br>
>>> pushing, considering, that there are two T-b tags to be added anyway?<br>
>><br>
>> I'll add it on my side before pushing the patch.<br>
> <br>
> Thanks a lot!<br>
> <br>
> Cheers,<br>
> Kai<br>
> <br>
> <br>
> <br>
</div></div><div class="HOEnZb"><div class="h5">> ______________________________<wbr>_________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
> <br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">-- <br>
<br>
Kai Wasserbäch (Kai Wasserbaech)<br>
<br>
E-Mail: <a href="mailto:kai@dev.carbon-project.org">kai@dev.carbon-project.org</a><br>
<br>
</font></span></blockquote></div><br></div>