[Lldb-commits] [PATCH] D42195: [lldb] Generic base for testing gdb-remote behavior

[Pavel Labath via lldb-commits]

On 22 January 2018 at 22:06, Jim Ingham <jingham at apple.com> wrote:
>
>
>> On Jan 22, 2018, at 3:10 AM, Pavel Labath via Phabricator <reviews at reviews.llvm.org> wrote:
>>
>> labath added subscribers: krytarowski, jingham, davide.
>> labath added a comment.
>>
>> In https://reviews.llvm.org/D42195#982035, @owenpshaw wrote:
>>
>>> - Added yaml2obj dependency.  It it okay to be an unconditional dependency?
>>
>>
>> Yeah, it probably needs to be conditional (if(TARGET yaml2obj)), because standalone builds will not have that target. OTOH, standalone builds will probably also have an issue with finding yaml2obj. However, I don't think it should be up to you to make standalone builds work. cc'ing Kamil and Michal, as they are the ones who use those builds.
>>
>> What I am not sure about is whether we need to make any changes to accomodate the XCode build. Jim, Davide: What would it take to make a test like this work from xcode?
>
> Sorry, I haven't been following.  These seem like standard lldb-unittest2 type tests.  What is the difficultly of getting such a test to run from Xcode?

The special thing about this test is that it uses the yaml2obj binary
from llvm. I'm not sure if it's really an issue (I don't have an xcode
build around), but I see a potential problem that this may fail
because it can't find the binary, either because it's not built, or
it's not in the right location.