<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Exchange Server">
<!-- converted from text --><style><!-- .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left: #800000 2px solid; } --></style>
</head>
<body>
<meta content="text/html; charset=UTF-8">
<style type="text/css" style="">
<!--
p
{margin-top:0;
margin-bottom:0}
-->
</style>
<div dir="ltr">
<div id="x_divtagdefaultwrapper" dir="ltr" style="font-size:12pt; color:#000000; font-family:Calibri,Helvetica,sans-serif">
<p>Hi Jonas,</p>
<p><br>
</p>
<p>Apologies for the non-build - I did a fetch/rebase this morning before creating the patch. Do I need to be on a different branch re StringRef vs ConstString? (Have seen that discussion in passing but confess I did not follow it in detail.) Any pointers
appreciated.</p>
<p><br>
</p>
<p>Re testing, I basically duplicated the test set from Lua and ran it using "ninja check-lldb-unit" and "ninja check-lldb-shell". I'm not really conversant with your testing infrastructure. Can you point me to something that might be suggest what I'd need
to do to generate a bot? I'll give it a shot if I have a starting point.</p>
<p><br>
</p>
<p>Also am happy to maintain the code in general and have a few folks in our project that can backstop me in case I get run over by a bus or something.
<span>😊</span></p>
<p><span><br>
</span></p>
<p><span>Best,</span></p>
<p><span>Dave</span></p>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> Jonas Devlieghere via Phabricator <reviews@reviews.llvm.org><br>
<b>Sent:</b> Wednesday, November 10, 2021 1:57:51 PM<br>
<b>To:</b> David Millar; anoronha@apple.com; fallkrum@yahoo.com; kkleine@redhat.com; teemperor@gmail.com; medismail.bennani@gmail.com; tedwood@quicinc.com; jmolenda@apple.com; syaghmour@apple.com; jingham@apple.com; vsk@apple.com; boris.ulasevich@gmail.com;
lldb-commits@lists.llvm.org; h.imai.833@nitech.jp; bruce.mitchener@gmail.com; david.spickett@linaro.org; quic_sourabhs@quicinc.com; djordje.todorovic@syrmia.com; serhiy.redko@gmail.com; Liburd1994@outlook.com<br>
<b>Cc:</b> mgorny@gentoo.org<br>
<b>Subject:</b> [PATCH] D111409: proposed support for Java interface to Scripting Bridge</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">JDevlieghere added a comment.<br>
<br>
Hi David, this looks really comprehensive. As far as the code is concerned, it has all the parts that we'd want to support another scripting language in LLDB. That leaves us with the last remaining questions:<br>
<br>
- Testing: having a bot that actually builds & tests this configuration.<br>
- Maintainership: having a commitment from you (or someone else) to maintain this code.<br>
<br>
Are you willing and able to sign up for those things?<br>
<br>
PS: I installed Java and applied your patch on top-of-tree. Things didn't built because of the recent change to the plugin manager that now expects `StringRef`s instead of `ConstString`s. Please let me know if you've rebased the patch and I'd love to give it
a try.<br>
<br>
<br>
<br>
================<br>
Comment at: lldb/cmake/modules/FindJavaAndSwig.cmake:13<br>
+ find_package(Java 11.0)<br>
+ find_package(JNI REQUIRED)<br>
+ if(JAVA_FOUND AND SWIG_FOUND)<br>
----------------<br>
This can't be `REQUIRED` because that will fail the CMake configuration stage when Java isn't found which is not what you want when doing autodetection (the default). Making it required is handled at a higher level for `FindJavaAndSwig` as a whole.
<br>
<br>
<br>
Repository:<br>
rLLDB LLDB<br>
<br>
CHANGES SINCE LAST ACTION<br>
<a href="https://reviews.llvm.org/D111409/new/">https://reviews.llvm.org/D111409/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D111409">https://reviews.llvm.org/D111409</a><br>
<br>
</div>
</span></font>
</body>
</html>