Issue with 3.1.1 android compiling

[kestrelm]

Issue with 3.1.1 android compiling

[kestrelm]

Fantastic! I think I might update glm for the godot runtime here in the future as well.

Cheers

[kestrelm]

Hello,

It looks like a compilation warning which results in the compiler refusing to go any further.
In the SCsub file of the godot plugin, you should add in the required additional compiler defines:

env_mymodule.Append(CPPDEFINES=['_USE_MATH_DEFINES', 'GLM_FORCE_RADIANS'])

to get rid of the warning. Give that a go.

[alperc84]

Thank you, that eliminated most of errors but we still can't continue:

[ 23%] �[94mCompiling �[95m==> �[93mmodules\creaturegodot\CreatureMetaData.cpp�[0m
=====
b"modules\\creaturegodot\\CreatureMetaData.cpp:102:34: warning: lambda capture 'this' is not used [-Wunused-lambda-capture]\r\n    auto getJsonNodeFromArray = [this](JsonNode& json_obj, int idx)\r\n                                 ^~~~\r\nmodules\\creaturegodot\\CreatureMetaData.cpp:518:31: warning: lambda capture 'this' is not used [-Wunused-lambda-capture]\r\n                                                                auto getColorFloat = [this](uint8_t val_in)\r\n                                                                                      ^~~~\r\nmodules\\creaturegodot\\CreatureMetaData.cpp:579:8: warning: unused variable 'radius' [-Wunused-variable]\r\n                auto radius = (float)morph_data.morph_res * 0.5f;\r\n                     ^\r\nIn file included from modules\\creaturegodot\\CreatureMetaData.cpp:1:\r\nIn file included from modules\\creaturegodot/CreatureMetaData.h:10:\r\nIn file included from modules\\creaturegodot\\glm\\gtx/../glm.hpp:110:\r\nIn file included from modules\\creaturegodot\\glm\\gtx/../gtc/../common.hpp:32:\r\nIn file included from modules\\creaturegodot\\glm/detail/func_common.hpp:470:\r\nmodules\\creaturegodot\\glm/detail/func_common.inl:720:12: error: use of undeclared identifier '_isnan'\r\n                                return _isnan(x) != 0;\r\n                                       ^\r\nmodules\\creaturegodot\\glm\\gtx/../gtc/packing.inl:143:16: note: in instantiation of function template specialization 'glm::isnan<float>' requested here\r\n                else if(glm::isnan(x))\r\n                             ^\r\nIn file included from modules\\creaturegodot\\CreatureMetaData.cpp:1:\r\nIn file included from modules\\creaturegodot/CreatureMetaData.h:10:\r\nIn file included from modules\\creaturegodot\\glm\\gtx/../glm.hpp:110:\r\nIn file included from modules\\creaturegodot\\glm\\gtx/../gtc/../common.hpp:32:\r\nIn file included from modules\\creaturegodot\\glm/detail/func_common.hpp:470:\r\nmodules\\creaturegodot\\glm/detail/func_common.inl:791:12: error: use of undeclared identifier '_isinf'\r\n                                return _isinf(x) != 0;\r\n                                       ^\r\nmodules\\creaturegodot\\glm\\gtx/../gtc/packing.inl:145:16: note: in instantiation of function template specialization 'glm::isinf<float>' requested here\r\n                else if(glm::isinf(x))\r\n                             ^\r\n3 warnings and 2 errors generated.\r\n"
=====
scons: *** [modules\creaturegodot\CreatureMetaData.android.opt.armv7.neon.o] Error 1
scons: building terminated because of errors.

btw my setup: godot-3.1.1-stable - android-sdk-tools-windows-4333796 - android-ndk-r20-windows-x86_64 - python-3.7.4 - pywin32-224.win32-py3.7 - scons-3.0.4 - jdk1.8.0_212 - Creature_Godot-master - vs2019

[kestrelm]

Hello,

Looks like those are all warnings which result from your very strict compiler settings. You can remove the 'this' keyword from the lambda bindings and recompile. As for the unidentified isNan and isInf calls, that is strange. You can take a look at this thread:
numworks/epsilon#115

For maybe more info. They suggest including math.h in the affected files. Or you can try replacing those isNan and isInf calls with the regular std versions. I don't have android so I am unable to verify but perhaps they have a different version of those math calls for their platform.

[alperc84]

Thank you for your time,
I updated GLM in "modules\creaturegodot\glm". https://github.com/g-truc/glm/releases/tag/0.9.9.6
and it's solved.