Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[R] Arrow package install - R binary version mismatch #37208

Closed
meztez opened this issue Aug 16, 2023 · 2 comments · Fixed by #37225
Closed

[R] Arrow package install - R binary version mismatch #37208

meztez opened this issue Aug 16, 2023 · 2 comments · Fixed by #37225

Comments

@meztez
Copy link
Contributor

meztez commented Aug 16, 2023

Describe the bug, including details regarding any error messages, version, and platform.

Error that you might see : *** Unable to find libcurl and openssl

For systems with multiple R versions installed (Posit Connect, Posit Workbench...).

Installing arrow could results in unexpected behavior.

Say you run install.packages("arrow") using /opt/R/4.3.1/bin/R but your system R version (which R) is /opt/R/4.3.0/bin/R.
Or maybe R executable is not on your path.

This script

runner <- "`R CMD config CXX17` `R CMD config CPPFLAGS` `R CMD config CXX17FLAGS` `R CMD config CXX17STD` -E -xc++"
will use the system's R instead of /opt/R/4.3.1/bin/R or R will not be found resulting in an error.

Would it be possible to locate the currently running R executable and use that instead?

runner <- "`R CMD config CXX17` `R CMD config CPPFLAGS` `R CMD config CXX17FLAGS` `R CMD config CXX17STD` -E -xc++"
r_exec <- paste(R.home(component = "bin"), "R CMD config", sep="/")
runner <- sprintf("`%s CXX17` `%s CPPFLAGS` `%s CXX17FLAGS` `%s CXX17STD` -E -xc++", r_exec, r_exec, r_exec, r_exec)

Thanks

Component(s)

R

@kou kou changed the title Arrow package install - R binary version mismatch [R] Arrow package install - R binary version mismatch Aug 17, 2023
@kou
Copy link
Member

kou commented Aug 17, 2023

Could you open a PR for this suggestion?

@thisisnic
Copy link
Member

Thanks for reporting this @meztez, sounds like a reasonable suggestion to me!

@kou kou added this to the 14.0.0 milestone Aug 28, 2023
kou added a commit that referenced this issue Aug 28, 2023
… (nix install) (#37225)

### Rationale for this change

See #37208.

### What changes are included in this PR?

Use the currrently running R binary to compile test program (nix install).

### Are these changes tested?

No. Installation.

### Are there any user-facing changes?

No

* Closes: #37208

Lead-authored-by: Bruno Tremblay <bruno@boostao.ca>
Co-authored-by: Sutou Kouhei <kou@cozmixng.org>
Co-authored-by: Neal Richardson <neal.p.richardson@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
loicalleyne pushed a commit to loicalleyne/arrow that referenced this issue Nov 13, 2023
…rogram (nix install) (apache#37225)

### Rationale for this change

See apache#37208.

### What changes are included in this PR?

Use the currrently running R binary to compile test program (nix install).

### Are these changes tested?

No. Installation.

### Are there any user-facing changes?

No

* Closes: apache#37208

Lead-authored-by: Bruno Tremblay <bruno@boostao.ca>
Co-authored-by: Sutou Kouhei <kou@cozmixng.org>
Co-authored-by: Neal Richardson <neal.p.richardson@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants