Bug 695381

What version of the ndk are you using? Supposedly versions < 5 declared, but did not define strtof. I'd advise you to build with a newer version. The ndk is at revision 10 at the time of writing...

I'm experiencing the same issue. I'm compiling on linux with ndk version 10. I'm only using armeabi-v7a. A week or 2 ago I compiled mupdf on windows and those versions worked fine.

That sounds like they broke the ndk again :(

Try adding the following to platform/android/jni/mupf.c:

#include <stdlib.h>

float strtof (const char* str, char** endptr)
{
    return (float)strtod(str, endptr);
}

Hopefully that should solve the problem. Please let us know if it does.

I got same issue and I add that method(strtof) at mupdf.c

but it still doesn't work

i use ndk-r10 on cygwin.

and i also have this follow problem when i'm doing ndk-build

---------------------------------------------------------------------------------
Android NDK: WARNING:jni/Android.mk:mupdfcore: LOCAL_LDLIBS is always ignored for static libraries
[armeabi-v7a] Compile thumb  : mupdfcore <= pdf-fontfile.c
jni/../../../source/pdf/pdf-fontfile.c:21:29: fatal error: gen_font_base14.h: No such file or directory
 #include "gen_font_base14.h"
                             ^
compilation terminated.
/cygdrive/e/android-ndk-r10/build/core/build-binary.mk:447: recipe for target 'obj/local/armeabi-v7a/objs/mupdfcore/__/__/__/source/pdf/pdf-fontfile.o' failed
make: *** [obj/local/armeabi-v7a/objs/mupdfcore/__/__/__/source/pdf/pdf-fontfile.o] Error 1
---------------------------------------------------------------------------------

please ignore following problem. my mistake.
anyway strtof issue still bother me

*** Bug 695398 has been marked as a duplicate of this bug. ***

(In reply to Ken Sharp from comment #6)
> *** Bug 695398 has been marked as a duplicate of this bug. ***

Hi @Ken, So is this bug not resolved yet?

(In reply to Jitesh Lalwani from comment #7)
> (In reply to Ken Sharp from comment #6)
> > *** Bug 695398 has been marked as a duplicate of this bug. ***
> 
> Hi @Ken, So is this bug not resolved yet?

If it was resolved the bug would be marked as such. I am not an Android developer, but it 'looks like' Google have broken the Android NDK again. But I don't know.

Hi Ken,

Thanks for the reply! Can we build using old Android NDK version? And will it create any problem as such?

(In reply to Jitesh Lalwani from comment #9)

> Thanks for the reply! Can we build using old Android NDK version? And will
> it create any problem as such?

As I said, I am not an Android developer, I can't answer this question.

NDK r10 works fine for me on Mac OS X.

Please could you try NDK r9d instead? You can download older versions of the NDK by editing the URL for the latest version and changing the version number.

Make sure you do an "ndk-build clean" before building after changing the version of the NDK.

Please let us know if this works for you.

(In reply to Matt Holgate from comment #11)
> NDK r10 works fine for me on Mac OS X.
> 
> Please could you try NDK r9d instead? You can download older versions of the
> NDK by editing the URL for the latest version and changing the version
> number.
> 
> Make sure you do an "ndk-build clean" before building after changing the
> version of the NDK.
> 
> Please let us know if this works for you.

Hi Matt thanks for replying but unfortunately changing the link to http://dl.google.com/android/ndk/android-ndk64-r9d-linux-x86_64.tar.bz2 does not download anything

I'll just try a linux myself build with ndk r10...

(In reply to Matt Holgate from comment #13)
> I'll just try a linux myself build with ndk r10...

Hi Matt,

I just now tried with older version ( http://dl.google.com/android/ndk/android-ndk-r9d-linux-x86_64.tar.bz2 ) but m still getting following error:

07-31 16:04:44.370: E/AndroidRuntime(12771): Process: com.artifex.mupdfdemo, PID: 12771
07-31 16:04:44.370: E/AndroidRuntime(12771): java.lang.UnsatisfiedLinkError: Couldn't load mupdf from loader dalvik.system.PathClassLoader[DexPathList[[zip file "/data/app/com.artifex.mupdfdemo-1.apk"],nativeLibraryDirectories=[/data/app-lib/com.artifex.mupdfdemo-1, /vendor/lib, /system/lib]]]: findLibrary returned null

This is odd. It works fine for me with the following configuration:

   - Linux (Ubuntu 10.04.4 LTS)
   - NDK r10 
   - SDK 16

I've tested the built APK on both a stock Nexus 5 running 4.4.4 and an emulator.

What kind of device are you testing on? I have seen these kind of problems with other projects on some HTC devices, although I never fully managed to get to the bottom of them.

Can you confirm if the build still fails when running in the emulator?

Thanks.

(In reply to Matt Holgate from comment #15)
> This is odd. It works fine for me with the following configuration:
> 
>    - Linux (Ubuntu 10.04.4 LTS)
>    - NDK r10 
>    - SDK 16
> 
> I've tested the built APK on both a stock Nexus 5 running 4.4.4 and an
> emulator.
> 
> What kind of device are you testing on? I have seen these kind of problems
> with other projects on some HTC devices, although I never fully managed to
> get to the bottom of them.
> 
> Can you confirm if the build still fails when running in the emulator?
> 
> Thanks.

Hi Matt,

I have tested on Moto G and Samsung tab. But I have a query here, if you were able to build on ubuntu, can you provide the source code files?

(In reply to Jitesh Lalwani from comment #16)

> I have tested on Moto G and Samsung tab. But I have a query here, if you
> were able to build on ubuntu, can you provide the source code files?

Do you mean the APK? I can certainly provide that.

(In reply to Matt Holgate from comment #17)
> (In reply to Jitesh Lalwani from comment #16)
> 
> > I have tested on Moto G and Samsung tab. But I have a query here, if you
> > were able to build on ubuntu, can you provide the source code files?
> 
> Do you mean the APK? I can certainly provide that.

No I mean Java classes as I have to use it as library i.e. mupdf/platform/android

No problem. I've sent you an email with a link to download the files.

(In reply to Matt Holgate from comment #19)
> No problem. I've sent you an email with a link to download the files.

Its working! Thanks a lot Matt, I wonder what was wrong in my build

Cool! Glad to hear it is working.

Did you try an ndk-build clean / ant clean btw?

(In reply to Matt Holgate from comment #21)
> Cool! Glad to hear it is working.
> 
> Did you try an ndk-build clean / ant clean btw?

Yes infact I deleted everything started from ground zero

Very weird. I'd be interested to know if you ever discover what the problem is!

(In reply to Matt Holgate from comment #23)
> Very weird. I'd be interested to know if you ever discover what the problem
> is!

Sure Matt. BTW I got one more crash. I just marked the code given by you as library and included in my project but it is crashing when I called as follows:

 Intent intent = new Intent(this, MuPDFActivity.class);

		 intent.setAction(Intent.ACTION_VIEW);

		 intent.setData(uri);

		 startActivity(intent);

java.lang.RuntimeException: Unable to start activity ComponentInfo{com.example.mupdfsample/com.artifex.mupdfdemo.MuPDFActivity}: android.view.InflateException: Binary XML file line #25: Error inflating class android.support.v7.internal.widget.ActionBarView

Ah, do you have the Android support library (http://developer.android.com/tools/support-library/index.html) added as a dependency of your project? It looks like we are trying to use it, but can't find its resources.

(In reply to Matt Holgate from comment #25)
> Ah, do you have the Android support library
> (http://developer.android.com/tools/support-library/index.html) added as a
> dependency of your project? It looks like we are trying to use it, but can't
> find its resources.

Hi Matt,

I have removed support for lower devices and it is working fine now :) Thanks for your great support!

Cool, no problem, thanks!

(In reply to Matt Holgate from comment #27)
> Cool, no problem, thanks!

Hi Matt,I again got that problem today:

08-04 18:45:37.520: E/AndroidRuntime(10157): java.lang.UnsatisfiedLinkError: Native method not found: com.c2cdroid.main.MuPDFCore.openFile:(Ljava/lang/String;)J

(In reply to Matt Holgate from comment #27)
> Cool, no problem, thanks!

I have just changed the package name

(In reply to Jitesh Lalwani from comment #29)
> (In reply to Matt Holgate from comment #27)
> > Cool, no problem, thanks!
> 
> I have just changed the package name

Then you need to change the packagename in platform/android/jni/mupdf.c too.

(In reply to Robin Watts from comment #30)
> (In reply to Jitesh Lalwani from comment #29)
> > (In reply to Matt Holgate from comment #27)
> > > Cool, no problem, thanks!
> > 
> > I have just changed the package name
> 
> Then you need to change the packagename in platform/android/jni/mupdf.c too.

Hi Matt,

I have changed com/artifex/mupdfdemo to com/c2cdroid/main but am still getting the crash. Do I need to compile somthing after making the change and how, since you have provided me the source code as it was building on my side?

Hi Matt,

I'm having the exact same issue as described here. Tried it in
- MAC OSX 10.9.3 & Archlinux 
- NDK r10 & r8b(earlist i could get)
- SDK 16

Had same issue across all environments. I see in the comments that Jitesh could make it work after you gave him the java files. It would be great if you could share the same with me. 

Thanks

Hi Matt,
Nevermind, Jitesh gave me the files. Thanks

Hi Avinash - cool.

It'd be good if we could fix the root cause though. I'm still unsure what is happening here :-(.

Hi Matt, Let me know if you need any info on the problem. I can send the entire project (that did not) to you, if you wish.

You need to use the NDK for 32-bit target platforms if you're targeting a 32-bit platform. The NDK for 64-bit target platforms builds broken binaries for 32-bit targets.

*** This bug has been marked as a duplicate of bug 695488 ***