ndk-build fails

[mrdevis]

Hi, Following your itinerary to build SC-Android, the instruction ndk-build fails with the linker complaining::

jni/Source/plugins/onsetsds.c:400: error: undefined reference to 'onsetsds_phase_rewrap' jni/Source/plugins/onsetsds.c:415: error: undefined reference to 'onsetsds_phase_rewrap' jni/Source/plugins/onsetsds.c:442: error: undefined reference to 'onsetsds_phase_rewrap' jni/Source/plugins/onsetsds.c:458: error: undefined reference to 'onsetsds_phase_rewrap' collect2: error: ld returned 1 exit status

This is on OSX10.10.5, crystax-ndk-10.3.1 Something known??

Thanks for any pointers!

Jeroen

[danstowell]

Seems weird. You'll notice that on line 38 of onsetsds.c, onsetsds_phase_rewrap is definitely defined: https://github.com/glastonbridge/SuperCollider-Android/blob/master/jni/Source/plugins/onsetsds.c#L38 So I can only presume your compiler is insane...

[mrdevis]

maybe has something to do with being inline?

[mrdevis]

... any cures known for an insane compile/linker??

[FRD93]

Hello everyone,

I've encountered the same issue (OSX Yosemite, crystax-ndk-10.3.1); I changed the lines

ifdef _WIN32

// the MSVC 2005 compiler is not up to date with inline float onsetsds_phase_rewrap(float phase); float onsetsds_phase_rewrap(float phase){ return (phase>MINUSPI && phase<PI) ? phase : phase + TWOPI * (1.f + floorf((MINUSPI - phase) * INV_TWOPI)); }

else

// Inline inline float onsetsds_phase_rewrap(float phase); inline float onsetsds_phase_rewrap(float phase){ return (phase>MINUSPI && phase<PI) ? phase : phase + TWOPI * (1.f + floorf((MINUSPI - phase) * INV_TWOPI)); }

endif

to this:

// FRD ADD /*

ifdef _WIN32

/ // END FRD ADD // the MSVC 2005 compiler is not up to date with inline float onsetsds_phase_rewrap(float phase); float onsetsds_phase_rewrap(float phase){ return (phase>MINUSPI && phase<PI) ? phase : phase + TWOPI * (1.f + floorf((MINUSPI - phase) \ INV_TWOPI)); }

// FRD ADD /*

else

// Inline inline float onsetsds_phase_rewrap(float phase); inline float onsetsds_phase_rewrap(float phase){ return (phase>MINUSPI && phase<PI) ? phase : phase + TWOPI * (1.f + floorf((MINUSPI - phase) * INV_TWOPI)); }

endif

*/ // END FRD ADD

and it worked. onsetsds.c.zip

[felixdollack]

I get the same error messages on macOS High Sierra 10.13.6 with crystax-ndk-10.3.2. The workaround from @FRD93 works fine for me as well.