wjones127 commented on code in PR #14273:
URL: https://github.com/apache/arrow/pull/14273#discussion_r983675689
##########
cpp/CMakePresets.json:
##########
@@ -117,17 +117,46 @@
"ARROW_GANDIVA": "ON"
}
},
+ {
+ "name": "features-python-minimal",
+ "inherits": [
+ "features-minimal"
+ ],
+ "hidden": true,
+ "cacheVariables": {
+ "ARROW_COMPUTE": "ON",
+ "ARROW_CSV": "ON",
+ "ARROW_FILESYSTEM": "ON",
+ "ARROW_JSON": "ON"
Review Comment:
I think I'm fine not including datasets if it's not required for the Python
build. In most workflows with CMakePresets.json, I think we expect developers
to create their own presets in CMakeUserPresets.json, which will inherit from
the provided once. (See [an example
here](https://www.datawill.io/posts/apache-arrow-development-cpp-conda-macos/#cmakeuserpresets.json))
Once this is merged, I can send a notice to the mailing list with
instructions on how to transition to the new presets. (I need to update my blog
post anyways.)
--
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]
