fix build against Android NDK (API Level 21 is required)
ClosedPublic

Authored by mgallien on Apr 27 2017, 7:32 PM.

Details

Summary

fix build against Android NDK (API Level 21 is required)

Test Plan

Test compilation under linux and Android

Diff Detail

Repository
R286 KFileMetaData
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
mgallien created this revision.Apr 27 2017, 7:32 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptApr 27 2017, 7:32 PM
Restricted Application added a subscriber: Frameworks. · View Herald Transcript
apol added a subscriber: apol.Apr 27 2017, 10:00 PM

+1 LGTM.

Maybe you can specify on the commit message that attr/xattr.h isn't present on the Android NDK and add a comment on the ifdef wouldn't hurt either.

mgallien updated this revision to Diff 13901.Apr 28 2017, 5:06 AM

added a comment and also modify the commit message

apol accepted this revision.May 8 2017, 10:43 AM
This revision is now accepted and ready to land.May 8 2017, 10:43 AM
This revision was automatically updated to reflect the committed changes.