Closed bac7d3ea-3f1b-4826-8464-f0b53d5e12d2 closed 14 years ago
Attachment: Solaris-fixes-for-R.patch.gz
Mercurial patch
Once this ticket is closed, so can #8272, as this addresses that issue.
Reviewer: Minh Van Nguyen
The updated R spkg builds on sage.math, bsd.math, rosemary.math, t2.math, Skynet machines (cleo and iras). However, it doesn't build on Cygwin (winxp1 on boxen.math). As of Sage 4.3.3.alpha1, the build on Cygwin fails during the compilation process of gd, which normally gets built before R. Also, I would note that #8191 is a prerequisite for this updated R spkg.
Merged r-2.10.1.p0.spkg in the standard spkg repository.
Merged: sage-4.3.4.alpha0
Build environment
The problem
Someone updated R in Sage, without testing on Solaris. The update caused several problems on Solaris - the most obvious one being R now needs iconv, which Sage does not currently include.
However, despite a new standard package for iconv #8191, R would still not build on Solaris, but failed with:
The Solution
Several changes were made to R's spkg-install file, to fix not only the fact R would not build at all, but also some other enhancements. The updated package can be found at
http://boxen.math.washington.edu/home/kirkby/Solaris-fixes/r-2.10.1.p0/
Adds the undocumented option
as that allows R to build without the ICU library from http://site.icu-project.org/ The option was suggested to me in a private email after a post to r-help r-project.org. The build now shows:
Removed all references to the option
as R must have iconv now - it is no longer optional. #8191 is an iconv library.
How to test
This should have no impact on any platform other than Solaris, though of course it is useful to test on multiple platforms to be 100% sure.
To test on Solaris 10, one needs access to a SPARC box. Some notes about how to test on 't2' are given at
http://wiki.sagemath.org/devel/Building-Sage-on-the-T5240-t2
First one needs to add a new standard package for iconv - #8191.
Once that is all down, typing 'make' should allow R to build, and you should get:
Upstream: None of the above - read trac for reasoning.
Component: porting: Solaris
Author: David Kirkby
Reviewer: Minh Van Nguyen
Merged: sage-4.3.4.alpha0
Issue created by migration from https://trac.sagemath.org/ticket/8285