-
Notifications
You must be signed in to change notification settings - Fork 630
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add lexer for Soong (Android.bp) config files #2659
Conversation
Just came back from travel, looking into this now. |
6c49b27
to
1a85eee
Compare
Thanks @Anteru. I fixed up the |
1a85eee
to
67edcd9
Compare
Thanks for the review, @Anteru! Your comment about whitespace helped me realize there was a deficiency in my previous version (multi-line lists weren't handled correctly). I've corrected this and added a test. Please take another look! |
67edcd9
to
c831dd4
Compare
b3b1e66
to
2318adf
Compare
2318adf
to
1bb1a50
Compare
Thanks @jeanas. All comments have been resolved. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Thanks!
This adds a lexer for
Android.bp
files which are used by the Soong Android build system.https://source.android.com/docs/setup/build
Here is an example of a full
Android.bp
file:platform/frameworks/base/Android.bp
.