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

remove dependency on `downloader` #203

Closed
achubaty opened this Issue Jul 7, 2015 · 1 comment

Comments

Projects
None yet
2 participants
@achubaty
Copy link
Contributor

achubaty commented Jul 7, 2015

R 3.2.1 should now be able to handle https:// on Windows by default (see http://developer.r-project.org/blosxom.cgi/R-devel/NEWS/2015/07/07#n2015-07-07). However, we can't expect all users to be up-to-date with their R versions (especially Windows users).

Until then, we can not import, and simply copy/paste the code from downloader:

My downloader package was created to deal with https on various platforms:
http://cran.r-project.org/web/packages/downloader/index.html

The package dates back to before R had libcurl support. It's very
simple: if you're on Windows, it uses setInternet2(), which enable
https support, and if you're on non-Windows, it looks for wget, curl,
and then lynx.

You don't have to import downloader in order to do this - there's just
one function you'd need, which, as far as I'm concerned, you can put
in your code:
https://github.com/wch/downloader/blob/master/R/download.r

If you need the code to officially relicensed, feel free to contact me by email.

-Winston

https://stat.ethz.ch/pipermail/r-package-devel/2015q3/000200.html

The downloader package is GPL-2 (https://github.com/wch/downloader/blob/master/DESCRIPTION)

@achubaty achubaty self-assigned this Jul 7, 2015

@eliotmcintire

This comment has been minimized.

Copy link
Contributor

eliotmcintire commented Nov 6, 2015

Sidestepped with change to require R 3.2.2 for SpaDES package

eliotmcintire added a commit that referenced this issue Nov 6, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.