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

Minetest.apk build failure #4005

Closed
Wayward1 opened this issue Apr 18, 2016 · 33 comments
Closed

Minetest.apk build failure #4005

Wayward1 opened this issue Apr 18, 2016 · 33 comments
Labels
Android Bug Issues that were confirmed to be a bug @ Build CMake, build scripts, official builds, compiler and linker errors

Comments

@Wayward1
Copy link
Contributor

Wayward1 commented Apr 18, 2016

For a while now, all my attempts to build the Minetest apk have met with this error: http://paste.ubuntu.com/15906887/. Also, here is the config.log file referenced within:
config.log.txt.

I have followed the instructions on the dev wiki exactly, on not only my main OS (Linux Mint 17.3 x64) but several VMs as well. (On a somewhat related note, it may be worth noting that I have to manually search for and install both the libgd2-xpm and ia32-libs-multiarch packages, as apt-get returns no results.) Have there been any recent changes not reflected on the wiki, or is there something else I may be missing?

@paramat paramat added @ Build CMake, build scripts, official builds, compiler and linker errors Android @ Documentation Bug Issues that were confirmed to be a bug labels Apr 18, 2016
@nerzhul
Copy link
Member

nerzhul commented Apr 18, 2016

@Wayward1 didn't notice this problem in my APK build, but i'm using Curl 7.48.0

@paramat
Copy link
Contributor

paramat commented Apr 20, 2016

Blocker for release?

@Zeno-
Copy link
Contributor

Zeno- commented Apr 20, 2016

I cannot build on Ubuntu unless I revert 096a01f ... so either the documentation needs updating or it should be fixed or it should be reverted before release

@Zeno-
Copy link
Contributor

Zeno- commented Apr 20, 2016

Works on Fedora 21 and 23

I wonder if Ubuntu and Debian are somehow using the wrong Android NDK version (i.e. because it adds a newer version of the toolchains to $PATH and it's using the wrong compiler)?

@paramat paramat added this to the 0.4.14 milestone Apr 20, 2016
@nerzhul
Copy link
Member

nerzhul commented Apr 20, 2016

@Zeno- if you find the dependency problem could be good. Yes blocker for me for release

@nerzhul nerzhul added Blocker The issue needs to be addressed before the next release. and removed @ Documentation labels Apr 20, 2016
@nerzhul
Copy link
Member

nerzhul commented Apr 20, 2016

Please note i don't have problems with building curl 7.48.0 for Android.

@Wayward1
Copy link
Contributor Author

It still fails for me with cURL 7.48.0

@Zeno-
Copy link
Contributor

Zeno- commented Apr 21, 2016

Have comments from this issue been deleted?

@Wayward1
Copy link
Contributor Author

Wayward1 commented Apr 21, 2016

Nope, not to my knowledge. I receive notifications for each comment

@est31
Copy link
Contributor

est31 commented Apr 21, 2016

@Wayward1 can you perhaps try to bisect? If its a regression, it can be easily found that way.

@Zeno-
Copy link
Contributor

Zeno- commented Apr 21, 2016

@est31 Warr bisected it but I don't know why his comments are not here anymore :/

@Warr1024
Copy link
Contributor

Warr1024 commented Apr 21, 2016

On Debian 8 (Linux 3.16.0-4-amd64 SMP Debian 3.16.7-ckt25-2 (2016-04-08) x86_64 GNU/Linux):

I was not able to get it to build, using either NDK r10d or r10e, and was getting the same error as above ("cannot run C compiled programs" in the curl dependency).

I bisected, and found the problem commit to be 096a01f. I was able to get it to build, using either NDK r10d or r10e, after reverting that one commit.

@est31
Copy link
Contributor

est31 commented Apr 22, 2016

I've split up 096a01f into four smaller commits in this branch: https://github.com/est31/minetest/commits/4005-testing

@Warr1024 has tested the commits, and the update openssl commit is the first one that doesn't build anymore.

@paramat
Copy link
Contributor

paramat commented Apr 23, 2016

@MoNTE48 are you able to help with this?

@MoNTE48
Copy link
Contributor

MoNTE48 commented Apr 23, 2016

sudo apt-get install m4

@Wayward1
Copy link
Contributor Author

Wayward1 commented Apr 24, 2016

That's included in the build instructions

@est31
Copy link
Contributor

est31 commented Apr 24, 2016

@Wayward1 can you test whether the last commit fixes the bug:
https://github.com/est31/minetest/commits/master

@MoNTE48
Copy link
Contributor

MoNTE48 commented Apr 24, 2016

Strange. Please, make clean_all && make openssl and write error, I want to know which package is missing on your system.

@Wayward1
Copy link
Contributor Author

Wayward1 commented Apr 24, 2016

@MoNTE48
Copy link
Contributor

MoNTE48 commented Apr 24, 2016

@Wayward1, Fuck...
pls, make clean_all && make gmp && make openssl

@Wayward1
Copy link
Contributor Author

@MoNTE48
Copy link
Contributor

MoNTE48 commented Apr 24, 2016

@Wayward1, "creating ../libssl.a" = build success.
make release -j4 and you get apk

@MoNTE48
Copy link
Contributor

MoNTE48 commented Apr 24, 2016

@est31, https://github.com/MultiCraftProject/MultiCraft/blob/master/build/android/Makefile#L306
:)
This is a mistake of the one who added gmp. This is your bug 181f7ba

@Wayward1
Copy link
Contributor Author

Cool, it indeed built successfully :D

@MoNTE48
Copy link
Contributor

MoNTE48 commented Apr 24, 2016

Have you spent on a solution to this stuff seven days. Guys, I'm really sorry for you.
@Wayward1, a word of advice, when you use the -j *, reading logs becomes a pain.

@paramat
Copy link
Contributor

paramat commented Apr 25, 2016

^ @est31
MoNTE thanks much appreciated.

@Wayward1
Copy link
Contributor Author

Yes, thank you MoNTE48 :)

@paramat paramat removed the Blocker The issue needs to be addressed before the next release. label Apr 25, 2016
@paramat
Copy link
Contributor

paramat commented Apr 25, 2016

@MoNTE48 any advice on how we fix this?

@MoNTE48
Copy link
Contributor

MoNTE48 commented Apr 25, 2016

@paramat, I'm glad I could help. :) I wrote the above link to line in multicraft, where correction

@MoNTE48
Copy link
Contributor

MoNTE48 commented Apr 25, 2016

#4033

@paramat
Copy link
Contributor

paramat commented Apr 25, 2016

@Wayward1 please could you test #4033 ?

@Wayward1
Copy link
Contributor Author

@paramat, will do

@paramat
Copy link
Contributor

paramat commented Apr 26, 2016

#4033 is merged.

@paramat paramat closed this as completed Apr 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Android Bug Issues that were confirmed to be a bug @ Build CMake, build scripts, official builds, compiler and linker errors
Projects
None yet
Development

No branches or pull requests

7 participants