nealrichardson commented on issue #35049: URL: https://github.com/apache/arrow/issues/35049#issuecomment-1523531216
I've dug into the configure script a bit in https://github.com/apache/arrow/pull/35147 and have a few thoughts: * We should remove the special `brew` case. Seems it would only be used if you have `apache-arrow` installed via brew but don't have `pkg-config`. If you want to use the brew libraries, either install `pkg-config` or specify `ARROW_HOME=$(brew --prefix apache-arrow)`. * If we use pkg-config to find libraries, we should do a strict version check like we do in other build scripts (allowing x.y.z.1 to match x.y.z but disallowing x.y.z.9000) and reject using the found system libraries if they don't match, rather than just warn. Set `ARROW_USE_PKG_CONFIG=true` to use them anyway. These don't solve the developer experience from a checkout entirely, but they eliminate some guns pointed at our feet. nixlibs.R will download a nightly binary if you're on .9000, but that's just for linux, and even then there are issues: depending on how you install, it may download every time; if it doesn't download every time, we'll need to provide instruction on how to keep it up to date if you contribute again in the future. On mac we don't host nightly libarrow binaries (bottles), so we can't just use them. The out of the box developer experience should just work, with no more brew conflicts, but it would be slow. -- 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]
