[ARROW-15185] Arrow R doesn't build correctly if Arrow C++ was configured with lowercase flags

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 7.0.0
• Component/s: R
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/30687
• Assignee: Alessandro Molina
• Reporter: Alessandro Molina
• Created: 22/Dec/21 11:05
• Updated: 11/Jan/23 08:45
• Resolved: 30/Dec/21 20:38

Description

I use to configure ARROW C++ using lowercase flag names "on" instead of "ON", that works fine with the C++ library and Python library, but leads to build errors on R.

That's because the R build process looks for compile flags in ArrowOptions.cmake and those compile flags are compared with the uppercase ones.

So in my ArrowOptions.cmake I had

### Build the Parquet libraries
set(ARROW_PARQUET "on") 

while the R build script was looking for

grep 'set(ARROW_PARQUET "ON")' $ARROW_OPTS_CMAKE 

this can be addressed by making those greps case insensitive

Issue Links

• links to: GitHub Pull Request #12017