Skip to content
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

Doesn't work on Android Emulator x86_64 #7

Open
Kaned1as opened this issue Nov 4, 2018 · 11 comments
Open

Doesn't work on Android Emulator x86_64 #7

Kaned1as opened this issue Nov 4, 2018 · 11 comments
Assignees
Labels
bug Something isn't working

Comments

@Kaned1as
Copy link

Kaned1as commented Nov 4, 2018

Surprisingly, this doesn't seem to be working on Android Emulator x86_64, but works normally on real phone with aarch64.

Update: checked armv7 and x86 emulators, works there too. Seems like only x86_64 libraries are affected.

Context:
OS: tried on Android 7.0 and Android 8.1
Versions: tried both 1.0.1 and 1.0.2b
Languages: tried on Java and Kotlin
Architectures: works on armv7, aarch64, x86 but doesn't work on x86_64

@geecko86 @svenoaks please take a look

@Kaned1as Kaned1as changed the title Doesn't work on Android Emulator Doesn't work on Android Emulator x86_64 Nov 4, 2018
@geecko86
Copy link
Contributor

geecko86 commented Nov 5, 2018

Hi,

I'm gonna need more info.
Which release are you running? Did you compile it yourself? Which version of Android? etc.

I just ran a unit test on a x86_64 emulator and fpcalc works as intended.

@Kaned1as
Copy link
Author

Kaned1as commented Nov 5, 2018

Hi geecko!

I didn't compile it myself. I tried releases 1.0.1 and 1.0.2b from this repo.

Versions: tried on Android 7.0 and Android 8.1
Languages: tried on Java and Kotlin

image

I just ran a unit test on a x86_64 emulator and fpcalc works as intended.

you mean, instrumented test, right? unit-tests run on your host system.

@Kaned1as
Copy link
Author

Kaned1as commented Nov 5, 2018

Tried on x86 KitKat and it works.

Maybe Google broke something again, like text relocations after Lollipop?

@geecko86
Copy link
Contributor

geecko86 commented Nov 5, 2018

implementation 'com.github.QuickLyric:fpcalc-android:1.0.2b'

This is what we use in production, including on x86 emulators.
If those binaries don't work for you I can't really think of why...

@geecko86
Copy link
Contributor

geecko86 commented Nov 5, 2018

You are getting a system error explicitly saying something is wrong with the native libs, right?

@Kaned1as
Copy link
Author

Kaned1as commented Nov 5, 2018

No, it just spits out nulls.
I'm using the same version.
I'm using emulators without Google APIs, can this be the reason?

Since you're here, can we hang out in IRC or Matrix or Signal or Telegram or Skype maybe? I'm all for digging up the cause.

@geecko86
Copy link
Contributor

geecko86 commented Nov 5, 2018

No I'm going out tonight (and also I'm not handling support for this repo).

But from the looks of it, it doesn't seem to be an issue with the binaries, rather with how you're using them. Perhaps it can't find the path you're giving it?

@Kaned1as
Copy link
Author

Kaned1as commented Nov 5, 2018

Perhaps it can't find the path you're giving it?

No. In that case it says ERROR in the output.
The file is totally there. Besides it works in armv7 and aarch64 with exact same paths.

@Kaned1as
Copy link
Author

Kaned1as commented Nov 5, 2018

Okay @geecko86 , thanks for your help anyway.
I collected all the facts we dug out in the initial bug description.

@geecko86
Copy link
Contributor

geecko86 commented Dec 26, 2018

I have actually been able to reproduce this issue recently. Seems like I was actually using a x86 emulator on a x86_64 system when I first replied to you.

Sorry!

I will investigate this when I have some more time.
In the mean time, a temporary workaround is to exclude all the x86_64 binaries in your build.gradle files.
Using x86 binaries on the x86_64 emulator works fine.

I did just that in 42b4fde.

@geecko86 geecko86 self-assigned this Dec 26, 2018
@geecko86 geecko86 added the bug Something isn't working label Dec 26, 2018
@Kaned1as
Copy link
Author

Thanks! I'm already using it in my plugin for Vanilla Music so it'd be perfect!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants