raulcd commented on code in PR #43407: URL: https://github.com/apache/arrow/pull/43407#discussion_r1691201336
########## cpp/src/arrow/vendored/musl/strptime.c: ########## @@ -18,8 +18,10 @@ #undef HAVE_LANGINFO #ifndef _WIN32 +# if !(defined(__ANDROID__) && __ANDROID_API__ < 26) Review Comment: that's a good point, we can remove this patch and probably remove it also upstream on the next release. It shouldn't be necessary as unsupported version, I suppose if we add an Android CI job might be worth validating -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
