[GitHub] [arrow] kou commented on a change in pull request #8813: ARROW-10780: [R] Update known R installation issues for CentOS 7

[GitBox <gi...@apache.org>]

kou commented on a change in pull request #8813:
URL: https://github.com/apache/arrow/pull/8813#discussion_r533697120



##########
File path: r/vignettes/install.Rmd
##########
@@ -284,7 +284,11 @@ so that we can attempt to improve the script.
 ## Known installation issues
 
 * On CentOS, if you are using a more modern `devtoolset`, you may need to set
-the environment variables `CC=/usr/bin/gcc CXX=/usr/bin/g++`.
+the environment variables `CC` and `CXX` (e.g., in the R `Makeconf` file).
+These may be set to system defaults (e.g., `CC=/usr/bin/gcc CXX=/usr/bin/g++`)
+or they may be set to use the `gcc`/`g++` from the devtoolset. In the latter
+case, you will have to make sure that the system arrow dependencies are not
+used (i.e., make sure the system variable `ARROW_USE_PKG_CONFIG` is false).

Review comment:
       Yes.
   We use devtoolset only on CentOS 6 because the system GCC on CentOS 6 is too old to build Apache Arrow C++. CentOS 7 or higher use the system GCC because they aren't old to build Apache Arrow C++.
   
   FYI: I'll dropped support for CentOS 6 soon because CentOS 6 reaches EOL.




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org