<div dir="ltr">Hi Charles,<br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 4, 2013 at 8:19 PM, Charles Davis <span dir="ltr"><<a href="mailto:cdavis5x@gmail.com" target="_blank">cdavis5x@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><br><div><div><div>On Apr 4, 2013, at 3:34 PM, Filipe Cabecinhas wrote:</div>
<br><blockquote type="cite"><div dir="ltr">Hi all,<div><br></div><div>I've changed the Makefiles around to compile the debugserver binary using them.</div></div></blockquote></div>So did I ;).<div><br><blockquote type="cite">
<div dir="ltr"><div>I moved debugserver.cpp and debugserver-entitlements.plist from source/... onto driver/...</div></div></blockquote></div>I think this is a good idea. Wish I'd thought of that...</div></div></blockquote>
<div>I couldn't see any other way to do it with llvm's Makefile build system.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">
<div>Later on, maybe we can make this a normal LLDB library, and/or split it into several different libraries.<br></div></div></blockquote><div>There's lots of abstracting to do, there. debugserver.cpp and the whole DNB layer pretty much scream “Mac OS X only” (Mach stuff + BSD stuff).</div>
<div><br></div><div>The best plan would be to make a new, plugin based, lldb-server (or something), in the regular LLDB sources, and create a binary like lldb, with some plugins hard-coded depending on compile-time feature detection. Greg Clayton has mentioned that in the mailing list, but it will take time. I suppose people will want to do that when they want to enable lldb remote debugging on Linux/FreeBSD/whatever, for example.</div>
<div><br></div><div>Since we need to rewrite debugserver, we might as well abstract it and make it plugin-based. ;-)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><div><div>I fixed the references I could find for those two files in the Xcode project file (both on the project files and on the entitlements for debugserver on iOS).<br><blockquote type="cite">
<div dir="ltr"><div><br></div><div>I brought in <a href="http://generate-vers.pl/" target="_blank">generate-vers.pl</a> from lldb/scripts to create debugserver_vers.c for Mac OS X (we could in the future abstract the script and keep only one copy, or use the same version numbering as clang and lldb-linux).</div>
</div></blockquote></div>IMO, unless we're being built Apple-style (i.e. with the Xcode project), we really should use the same versioning scheme as Clang and LLDB on Linux. But that's just my opinion.</div></div>
</blockquote><div>That could be, but I prefer to have the version numbers done in the same way. Either we also change lldb's version number stuff (it's also using <a href="http://generate-vers.pl" target="_blank">generate-vers.pl</a> on Mac OS X + Makefiles), or we keep the script on both.</div>
<div><br></div><div>I don't mind getting rid of the script and just use the same versioning scheme as on Linux, but that would make different code execute depending on the build system, which I'm not very fond of seeing.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div>I also added Makefiles for the subdirectories under debugserver/source, of course.<br>
<blockquote type="cite"><div dir="ltr"><div><br></div><div>Apple folks: Could you check out the xcodeproj files and confirm that everything is okay with you?</div>
<div>Linux/FreeBSD/others: Please test it to see what changes may be needed to not break the build. I'm thinking about only including debugserver in DIRS if uname -s is Darwin.</div></div></blockquote></div>That's what I did in mine.</div>
<div><br></div><div>Hang on, maybe I can see about putting our patches together. Mine has some other stuff (e.g. dependency tracking for the MIG source, universal build support) that yours doesn't have.</div></div></blockquote>
<div>Please send me the patches, then. I will incorporate before committing.</div><div>About the dependency tracking: I added the DNBConfig.h dependency, now.</div><div><br></div><div style>Thanks,</div><div style><br></div>
<div style> Filipe</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><div><br></div><div>Chip</div><div><br><blockquote type="cite">
<span><lldb-makefiles.patch></span>_______________________________________________<br>lldb-commits mailing list<br><a href="mailto:lldb-commits@cs.uiuc.edu" target="_blank">lldb-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
</blockquote></div><br></div></blockquote></div><br></div></div>