You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "thisisnic (via GitHub)" <gi...@apache.org> on 2023/06/02 04:58:19 UTC

[GitHub] [arrow] thisisnic opened a new issue, #35884: [R] Developer setup guides need more context on SSL versions

thisisnic opened a new issue, #35884:
URL: https://github.com/apache/arrow/issues/35884

   ### Describe the enhancement requested
   
   In the [R dev setup guides](https://arrow.apache.org/docs/r/articles/developers/setup.html), it says:
   
   > The directory names correspond to the OpenSSL version the binaries built with: - “linux-openssl-1.0” (OpenSSL 1.0) - “linux-openssl-1.1” (OpenSSL 1.1) - “linux-openssl-3.0” (OpenSSL 3.0)
   
   We need to add context about which one to choose - there's nothing here about that at all.
   
   ### Component(s)
   
   R


-- 
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: issues-unsubscribe@arrow.apache.org.apache.org

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

[GitHub] [arrow] assignUser commented on issue #35884: [R] Developer setup guides need more context on SSL versions

Posted by "assignUser (via GitHub)" <gi...@apache.org>.
assignUser commented on issue #35884:
URL: https://github.com/apache/arrow/issues/35884#issuecomment-1577208072

   I think it would be nice to have a way to get the latest nightly binary (even if the version is *.9000) + a caveat that it might not work with a more recent checkout?


-- 
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: github-unsubscribe@arrow.apache.org

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

[GitHub] [arrow] thisisnic commented on issue #35884: [R] Developer setup guides need more context on SSL versions

Posted by "thisisnic (via GitHub)" <gi...@apache.org>.
thisisnic commented on issue #35884:
URL: https://github.com/apache/arrow/issues/35884#issuecomment-1576867554

   We should update the docs to mention what is on-label then.
   
   From chatting with Dewey earlier, it looks like the "simple path" for Linux/MacOS is basically:
   * Clone arrow
   * Open the Arrow.proj file in the R directory
   * Run `devtools::load_all()` (which will but subsequently it doesn't)
   
   + for Windows, still use the nightlies
   


-- 
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: github-unsubscribe@arrow.apache.org

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

[GitHub] [arrow] nealrichardson commented on issue #35884: [R] Developer setup guides need more context on SSL versions

Posted by "nealrichardson (via GitHub)" <gi...@apache.org>.
nealrichardson commented on issue #35884:
URL: https://github.com/apache/arrow/issues/35884#issuecomment-1573765116

   > we have a list of them somewhere for the purposes of installing the correct release binary
   
   I don't think we do anymore--we just check the openssl version: https://github.com/apache/arrow/blob/main/r/tools/nixlibs.R#L184-L193
   
   Even though it's listed in this doc, I'd still consider using the nightly libarrow binaries for local package development to be "off-label", it's not what they're intended for. But I think adding a sentence to call `openssl version` to see which one you should use is fine for these purposes. 


-- 
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: github-unsubscribe@arrow.apache.org

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

[GitHub] [arrow] paleolimbot commented on issue #35884: [R] Developer setup guides need more context on SSL versions

Posted by "paleolimbot (via GitHub)" <gi...@apache.org>.
paleolimbot commented on issue #35884:
URL: https://github.com/apache/arrow/issues/35884#issuecomment-1573747238

   I think the incantation to check for the openssl version that the system package manager has installed is `openssl version`...the commands to install it are provided (perhaps confusingly) later on in the developer guide. We could also just list distributions that commonly use one or the other...we have a list of them somewhere for the purposes of installing the correct release binary.
   
   After @nealrichardson's PR refactoring the configure script, a totally reasonable approach is also just a naive "install" or "load all", which will take forever the first time while it builds libarrow but won't attempt to *re*build libarrow unless a user explicitly deletes the libarrow directory.


-- 
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: github-unsubscribe@arrow.apache.org

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