Hello Jan and Kai,

would be nice if this lands (have T-b from me).

Dieter

Am 13.05.2018 19:10, schrieb Jan Vesely:
Hi,

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.

Jan

On Sun, May 13, 2018 at 3:56 AM, Kai Wasserbäch
<k...@dev.carbon-project.org> wrote:

Ping! Can somebody *please* commit this patch? It fixes an FTBFS,
has two T-b
and one A-b.

Kai Wasserbäch wrote on 07.05.2018 16:48:
Jan Vesely wrote on 02.05.2018 22:38:
On Wed, 2018-05-02 at 18:38 +0200, Kai Wasserbäch wrote:
[...]

Thank for looking into this. We probably need CLANG_LIBS handling
similar to LLVM_LIBS. I agree this is the best fix for now.

Acked-by: Jan Vesely <jan.ves...@rutgers.edu>

libclang.so might be a solkution, but I'm not sure how it
interacts
with older or static build clang. It's also weird that we are
linking
to clang here instead of clover which actually uses clang
symbols.

@Emil, are you OK with this patch?

Gentle ping.

-lclangDriver \
-lclangSerialization \
-       -lclangCodeGen \

Is this change related?

Not really, just a minor clean-up while I was busy a few lines
above.
"clangCodeGen" is already named on the first Clang library
line.

ah, all right, maybe mention it in the commit message?

Do I need to resend the patch for that or can you just add a
line like "This
change also removes the duplicate clangCodeGen line (trivial
change)." before
pushing, considering, that there are two T-b tags to be added
anyway?

I'll add it on my side before pushing the patch.

Thanks a lot!

Cheers,
Kai




_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev [1]


--

Kai Wasserbäch (Kai Wasserbaech)

E-Mail: k...@dev.carbon-project.org



Links:
------
[1] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to