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

Upgrade R to 3.4.2 #24026

Closed
EmmanuelCharpentier mannequin opened this issue Oct 12, 2017 · 8 comments
Closed

Upgrade R to 3.4.2 #24026

EmmanuelCharpentier mannequin opened this issue Oct 12, 2017 · 8 comments

Comments

@EmmanuelCharpentier
Copy link
Mannequin

EmmanuelCharpentier mannequin commented Oct 12, 2017

The reasons are the same as usual (getting help on r-help almost requires an up-to-date interpreter.

The original tarball at https://cran.r-project.org/src/base/R-3/R-3.4.2.tar.gz .

CC: @jdemeyer @embray @dimpase

Component: packages: standard

Keywords: r-project

Author: Emmanuel Charpentier

Branch/Commit: f6a8c89

Reviewer: Erik Bray

Issue created by migration from https://trac.sagemath.org/ticket/24026

@EmmanuelCharpentier EmmanuelCharpentier mannequin added this to the sage-8.1 milestone Oct 12, 2017
@EmmanuelCharpentier
Copy link
Mannequin Author

EmmanuelCharpentier mannequin commented Oct 13, 2017

Branch: u/charpent/upgrade_r_to_3_4_2

@EmmanuelCharpentier
Copy link
Mannequin Author

EmmanuelCharpentier mannequin commented Oct 13, 2017

Commit: f6a8c89

@EmmanuelCharpentier
Copy link
Mannequin Author

EmmanuelCharpentier mannequin commented Oct 13, 2017

Author: Emmanuel Charpentier

@EmmanuelCharpentier
Copy link
Mannequin Author

EmmanuelCharpentier mannequin commented Oct 13, 2017

comment:2

First attempt :

  • installed the upstream tarball un $SAGE_ROOT/upstream ;
  • updated version numbers and checksums :
  • rebased patches along the lines of the Developer's guide.

The resulting R :

  • passes ptestlong with no error whatsoever ;
  • sort-of passes its own test suite (i. e. a couple of expected, pre-announced, failures)

Patch rebasing : I followed the suggestions of the Developer's guide, but for each original patch individually, and manually merged the original patch's initial comment in the new patch.

I still have doubts about the legitimacy of libcurl_https_support.patch, which lift's R enforcement of https support. I am aware that Jeroen was sanguine about not enforcing the use of openssl, but do not know if he maintains this position in view of the recent (ongoing ?) sort:relevance/sage-devel/3aifUzpfKOw/yGjBhL6iCAAJ change of license. Furthermore, I think that switching as much as possible to authenticated Web sources is a Good Idea (TM), and that this move is almost irresistible ; I still don't like to fight the weather...

A usual, I think that this should be checked on out problem children of platforms :

  • Macintosh,
  • Windows on Cygwin (at least 64 bits).

Hence the Cc to Dimitri (who is, AFAICT, the most knowledgeable Sage developer about Macs and their shenanigans) and Erik (whose heroic port to Cygwin is a major boon to Sage).

needs_review


New commits:

f6a8c8924026 : rebased patches, updated version.

@dimpase
Copy link
Member

dimpase commented Oct 19, 2017

comment:3

works on OSX (10.12), and on FreeBSD. (Disclaimer --- I don't use OSX for work since many years already, I just still have an old and slow MacBook Air on which I can test things).

As OSX support on 10.12 is currently dependent on open tickets, this is a test done with clang as the main C compiler, using #12426,
and gfortran built using #23898.

@embray
Copy link
Contributor

embray commented Oct 30, 2017

Reviewer: Erik Bray

@embray
Copy link
Contributor

embray commented Oct 30, 2017

comment:4

LGTM on Cygwin

@vbraun
Copy link
Member

vbraun commented Nov 1, 2017

Changed branch from u/charpent/upgrade_r_to_3_4_2 to f6a8c89

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants