[cfe-commits] Support Android standalone toolchain in Clang driver

Evgeniy Stepanov eugeni.stepanov at gmail.com
Mon Sep 3 02:10:15 PDT 2012


Thanks. I've landed a rebased change as r163109.

For the record, the issue was not with .keep files.

A  +    test/Driver/Inputs/basic_android_tree/sysroot/usr
D       test/Driver/Inputs/basic_android_tree/usr

The above is "usr" being moved one level below under "sysroot". Svn
diff includes chunks for the deletion of everything under "usr", but
nothing about it being added back under "sysroot/usr'.


On Tue, Aug 28, 2012 at 9:19 PM, Rafael EspĂ­ndola
<rafael.espindola at gmail.com> wrote:
> On 28 August 2012 05:56, Evgeniy Stepanov <eugeni.stepanov at gmail.com> wrote:
>> Rafael, thanks for your consideration!
>>
>> I forgot to include the test for the C++ include dir in the latest
>> patch. Attaching a full patch this time.
>>
>> Believe it or not, but Inputs/basic_android_tree/sysroot does exist.
>> SVN is just not smart enough to show it in the patch. Here is what I
>> plan to commit, if approved:
>
> I remember that svn bug now :-(
>
> I think you will also need .keep file in any empty directory for the
> benefit of version control systems that don't handle empty
> directories. LGTM with that.
>
> Cheers,
> Rafael




More information about the cfe-commits mailing list