[Lldb-commits] [PATCH] D41902: Remove Platform references from the Host module

[Pavel Labath via lldb-commits]

On 11 January 2018 at 16:04, Greg Clayton <clayborg at gmail.com> wrote:
>
>> On Jan 11, 2018, at 5:12 AM, Pavel Labath <labath at google.com> wrote:
>>
>> Thanks for adding the test Jim. I have confirmed that it passes with
>> this patch applied (because the bundle is resolved during target
>> creating, not launching).
>>
>> However, this did make me aware of the fact that this patch removed
>> the bundle resolution logic from the launcher itself, so I'm going to
>> add back that part.
>
> The platform really does need a part in this. Each platform might have different files. APK files for Android, .ipa files for iTunes, .app bundles for all darwin operating systems and more. So the platform really is the right place for this kind of logic.

Yes, but this is in the host layer, which already assumes the
knowledge of the host platform -- there no sense in trying to "launch"
an APK file on iOS. It actually fits quite nicely, since the code that
actually resolved the executable in the bundle was already present in
the host layer.