-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Android TestApp Crashing When MapView Is Loaded #5108
Comments
@kkaefer Did you rebuild the Android geojsonvt binaries as part of mapbox/mason@72f30dd? If so, and if we don't have a quick fix for this, we should roll back that commit, and the binaries. |
I checked out the latest commit prior the NDK update in master ( 7272d7c ), updated the git submodule, made the SDK / TestApp, and successfully ran it on the Samsung S5. This tells me that @jfirebaugh is on the right path. |
Hmm, that's inconsistent with what you earlier reported @bleege. Building on 7272d7c should be exactly equivalent to building with the prior mason hash, because the only change in 25c4436 was updating the mason hash. The reason I said the binaries could be the issue is because you said that reverting mason didn't help. If the binaries changed in an incompatible manner, than reverting mason wouldn't completely resolve the issue, because the binaries are stored on S3. So we need to conclusively determine whether or not reverting 25c4436 resolves the issue, as you are now reporting, or whether it does not, as you reported in the initial description. |
Yeah, this is an odd one. I'll give things another try with git checkout 7272d7c84cf6e874b417ecd165f46f26ae7729c9
git submodule update
make clean
make android The original attempt to use the "old" Mason dependencies with the latest from git pull origin master // Equivalent to git checkout 25c4436d37d84cb8eb800385b412e784a0789087
cd .mason/
git checkout c03b19241ad6450dab16bf06466f849cb5a15757
cd ../
make clean
make android |
I just did a clean build from git checkout master
git pull // Grabs latest from `master` which is the NDK commit at 25c4436d37d84cb8eb800385b412e784a0789087
git submodule update // Results in Submodule path '.mason': checked out '72f30dd4a483f8e52133b8112c061db10c04fdba'
make clean
make android Here's the stack trace from this: 05-23 21:39:13.679 15095-15095/com.mapbox.mapboxsdk.testapp E/art: dlopen("/data/app/com.mapbox.mapboxsdk.testapp-1/lib/arm/libmapbox-gl.so", RTLD_LAZY) failed: dlopen failed: cannot locate symbol "_ZN6mapbox9geojsonvt7Convert11projectRingERKNSt6__ndk16vectorINS0_6LonLatENS2_9allocatorIS4_EEEEd" referenced by "libmapbox-gl.so"...
05-23 21:39:13.689 15095-15095/com.mapbox.mapboxsdk.testapp E/AndroidRuntime: FATAL EXCEPTION: main
Process: com.mapbox.mapboxsdk.testapp, PID: 15095
java.lang.UnsatisfiedLinkError: dlopen failed: cannot locate symbol "_ZN6mapbox9geojsonvt7Convert11projectRingERKNSt6__ndk16vectorINS0_6LonLatENS2_9allocatorIS4_EEEEd" referenced by "libmapbox-gl.so"...
at java.lang.Runtime.loadLibrary(Runtime.java:371)
at java.lang.System.loadLibrary(System.java:989)
at com.mapbox.mapboxsdk.maps.NativeMapView.<clinit>(NativeMapView.java:44)
at com.mapbox.mapboxsdk.maps.MapView.initialize(MapView.java:203)
at com.mapbox.mapboxsdk.maps.MapView.<init>(MapView.java:178)
at com.mapbox.mapboxsdk.testapp.activity.annotation.PolygonActivity.onCreate(PolygonActivity.java:55)
at android.app.Activity.performCreate(Activity.java:6221)
at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1119)
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2614)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2726)
at android.app.ActivityThread.access$900(ActivityThread.java:172)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1421)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:145)
at android.app.ActivityThread.main(ActivityThread.java:5835)
at java.lang.reflect.Method.invoke(Native Method)
at java.lang.reflect.Method.invoke(Method.java:372)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1399)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1194) |
Same here just after updating to new snapshot release.
|
Having the same issue as @mpuchala |
Discussed internally and decided to revert the Android NDK update commit 25c4436 for now so that development can continue. |
I reverted the Android NDK commit, rebuilt locally, and successfully tested the TestApp on a Samsung S5. The reverted commit is 6abd3d7 on The next step is to create a new ticket to upgrade to the latest NDK and start work on it after |
This has regressed on |
After updating to the latest on
master
25c4436 and running the TestApp on a Samsung S5 (API 21) the app crashes when any Activity with aMapView
is loaded. The error (see stacktrace below) points to an issue with thegeojsonvt
, and not the still open issue about switching arm64 to usegold
instead ofbfd
as the Samsung S5 is 32 bit. I've tried checking out the previous version ofmason
c03b19241ad6450dab16bf06466f849cb5a15757 and rebuilding with the same issue happening. Based on that it's unclear to me if this is related to the new NDK version in #5092 .Steps To Reproduce
make clean
make android
/cc @tmpsantos @kkaefer @jfirebaugh @zugaldia @tobrun @cammace @danswick
The text was updated successfully, but these errors were encountered: