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

Android - Compilation error #31

Closed
ceithir opened this issue Jan 6, 2014 · 5 comments
Closed

Android - Compilation error #31

ceithir opened this issue Jan 6, 2014 · 5 comments

Comments

@ceithir
Copy link

ceithir commented Jan 6, 2014

Hello,

I am having some problems compiling the Readium SDK for Android on Debian Wheezy.

First, I had errors about missing dependencies, caused by Linux case sensitivity (ePub3 is not epub3), but I managed to solve them easily with symlinks.

But now, I am still stuck with a new serie of errors:

./../../ePub3/utilities/CPUCacheUtils_arm.s:16: Error: junk at end of line, first unrecognized character is `\'
./../../ePub3/utilities/CPUCacheUtils_arm.s:27: Error: bad instruction `label(C_FN_NAME(epub_sys_cache_invalidate))'
./../../ePub3/utilities/CPUCacheUtils_arm.s:34: Error: bad instruction `label(C_FN_NAME(epub_sys_cache_flush))'

The stangest thing is that the last commit for this file states "Now builds on Android. Requires ARMv7.". But I use ARMv7, and it does not build.

Could you help me?

Thanks,
Éric

@rohinkumar
Copy link

I am facing the same issue. We had to change the case-insensitive file inclusions everywhere. After removing all of them, we are getting "make: *** [path/readium-sdk-master/Platform/Android/obj/local/armeabi-v7a/objs/epub3/ePub3/ePub/container.o] Error 1".

Both master and developer branches have the same issue. The earlier version we could compile was unable to display epub content. So in essence, we have no usable android library of readium. If you know any please suggest or resolve this issue.

@prcolaco
Copy link
Member

prcolaco commented Jan 7, 2014

Hi ceithir and rohinkumar,

Thanks for the bug report. In fact the support of the build system for platforms other than Windows (for Windows platform build) and Mac for other platform builds (iOS, OSX and Andorid) is limited, as stated in the readme file.

Although we have some hints on how to build the Android platform under Linux in the readme, they are quite old and outdated as I can see... Since most of us working on the Android platform use Mac, we have some difficulty in reproducing the problems you are facing right now.

This way I ask the Android contributors that have access to a Linux machine to give it a try and get some feedback in this thread. I'll also find some time to do it myself as well.

Can you please post here the versions of Host OS, Android SDK, Android NDK, other tools used, as well as if compilation was command line or in Eclipse, and other information you might find relevant?

Cheers, Pedro

@rohinkumar
Copy link

Thanks Pedro for quick response. I am using Ubuntu 12.04. using Android SDK 17 and NDK r9. I am using the standard ndk-compile.sh build /path/to/ndk on zsh shell as prescribed.

I will try to upload renamed files with updated path structure (see issue #32). But it would be of immense help if you can redirect me to a older working snapshot.

@ceithir
Copy link
Author

ceithir commented Jan 8, 2014

Thanks for your answer. For me, it's Debian 7 (64-bit), 20131030 version of sdk (64-bit - there is about 6 build tools in this, from 17 to 19.0.1, no idea which one is used), ndk r9c (64-bit).

Compiling with standard line, inside Eclipse (auto-configuration by importing .project):

$NDK_PATH/ndk-build -C . V=1 NDK_APPLICATION_MK=Application.mk NDK_PROJECT_PATH="$PROJECT_PATH" APP_BUILD_SCRIPT=Android.mk

I prepared the dependencies with ndk-compile.sh, as recommended.

Changes I had to do for case-sensitivity:

  • ln -s include/ePub3 include/epub3 (the proper way should be to fix all paths with the wrong case, but I was worried I could break something else by doing so)
  • In Android.mk, replace all *.S by *.s (three files only: ePub3/utilities/CPUCacheUtils_arm.s, ePub3/utilities/CPUCacheUtils_i386.s, ePub3/utilities/CPUCacheUtils_x64.s)

@rkwright
Copy link
Member

Closing this issue as obsolete. If any of the posters are still interested in this or have new info, please start a new issue. To repeat, we do not currently support Linux, but welcome any contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants