Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upConfigure the Android NDK #158
Conversation
|
Please submit these upstream, as well. |
| def stlport_cppflags(value, ndk, _): | ||
| if value and len(value): | ||
| - return value.split() | ||
| + return value[0].split() |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
asajeffrey
Oct 23, 2018
Author
Member
Good question. I made the edit to bring STLPORT_CPPFLAGS in line with the other configurations, which use value[0]. Building again without this change still works, so I'll undo it.
This comment has been minimized.
This comment has been minimized.
asajeffrey
Oct 23, 2018
Author
Member
Ah, try again, I was building with the wrong version. Without this change, we get:
File "/Users/ajeffrey/.cargo/git/checkouts/mozjs-ccfd509d897f834d/f3d2e03/mozjs/build/moz.configure/android-ndk.configure", line 276, in stlport_cppflags
return value.split()
AttributeError: 'PositiveOptionValue' object has no attribute 'split'
make: *** [maybe-configure] Error 1
|
I'll upstream these changes. @bors-servo r=jdm |
|
|
|
@bors-servo r- |
|
@bors-servo r=asajeffrey |
|
|
Configure the Android NDK This patch adds configuration for the Android NDK directory layout. It's used when building for the Magic Leap, which uses a very similar, but not quite identical, NDK. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/mozjs/158) <!-- Reviewable:end -->
|
|
|
@bors-servo retry CI weirdness. |
Configure the Android NDK This patch adds configuration for the Android NDK directory layout. It's used when building for the Magic Leap, which uses a very similar, but not quite identical, NDK. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/mozjs/158) <!-- Reviewable:end -->
|
|
asajeffrey commentedOct 19, 2018
•
edited by larsbergstrom
This patch adds configuration for the Android NDK directory layout. It's used when building for the Magic Leap, which uses a very similar, but not quite identical, NDK.
This change is