plf_colony icon indicating copy to clipboard operation
plf_colony copied to clipboard

PLF_CPP20_SUPPORT shouldn't be defined on Android NDK r24

Open mikezackles opened this issue 3 years ago • 3 comments
trafficstars

As best I can tell, this detection is broken for the Android NDK, which ships with a copy of libc++ that does not implement the concepts header. Maybe it would be appropriate to use _LIBCPP_VERSION? There doesn't appear to be a way for users to override this behavior without patching.

mikezackles avatar Apr 29 '22 21:04 mikezackles

For what it's worth, the relevant NDK issue is android/ndk#1530. They are shipping a libc++ that's something like 2 years out of date compared to the version of clang they're shipping.

mikezackles avatar May 20 '22 18:05 mikezackles

Hey Zachary, thanks for getting back to me on this, I'm currently busy on the hive (C++ standards version of colony) implementation, but once done there (probably a month to go) will do this. Cheers! Matt

On 21/05/2022 6:00 am, Zachary Michaels wrote:

For what it's worth, the relevant NDK issue is android/ndk#1530 https://github.com/android/ndk/issues/1530. They are shipping a libc++ that's something like 2 years out of date compared to the version of clang they're shipping.

— Reply to this email directly, view it on GitHub https://github.com/mattreecebentley/plf_colony/issues/46#issuecomment-1133173610, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABE4FIVBAIKIWGGCS7PC54LVK7HMXANCNFSM5UXIN6IA. You are receiving this because you are subscribed to this thread.Message ID: @.***>

mattreecebentley avatar May 23 '22 03:05 mattreecebentley

No problem -- no rush from me. Love the lib. Thanks!

mikezackles avatar May 23 '22 04:05 mikezackles

I think this is fixed now - please check for me-

mattreecebentley avatar Aug 27 '22 22:08 mattreecebentley