fread tries to use wget on file:// input in RStudio #1668

Activity

1. jangorecki commented on Apr 22, 2016

   

   jangorecki

   on Apr 22, 2016

   Last edited by jangorecki

   Member

   More actions

   My guess is that RStudio may silently change download.method option and affect output, comparing to base R.
   Anyway if it works in R, and just not works in RStudio then you should report it to RStudio. Base R should be the platform to which other projects should be conformed.

   As you are on linux you may find my 3 helpful scripts:

   # dt.build
   R CMD build data.table --no-manual
   # dt.install
   R CMD INSTALL data.table_1.9.7.tar.gz
   # dt.check
   R CMD check data.table_1.9.7.tar.gz --no-manual --as-cran
   

   Use chmod to make them executable, and call in shell from parent dir of data.table:

   ./dt.build && ./dt.install # install devel
   ./dt.build && ./dt.check # check devel
   

   Once I started to use them I had no need to debug issues related to intermediate helpers. It also plays nicely when switching to other branches.
2. MichaelChirico commented on Apr 23, 2016

   

   MichaelChirico

   on Apr 23, 2016

   MemberAuthor

   More actions

   Indeed, because I have the "Use secure download method for HTTP" option set in Tools>Global Options...>Packages, getOption("download.file.method", default = "auto") is set to wget, which doesn't support file:///.

   Seems like a fix would be to force curl in the case that is_file is true.

   Gonna put a PR to that end.
3. MichaelChirico

   changed the title [-][Internals] Tests 1378.2, 1378.3 fail in RStudio[/-] [+]fread tries to use wget on file:// input in RStudio[/+] on Apr 23, 2016
4. arunsrinivasan

   closed this as completedin 9fca25aon Apr 24, 2016
5. arunsrinivasan

   added a commit that references this issue on Apr 24, 2016

   Merge pull request #1670 from MichaelChirico/download_file

   ...

   This issue will close once commit 6873afa is merged into the 'master' branch.

   6873afa