[Rd] Latest R-devel build failing on OS X

[frederik at ofb.net]

Thanks. OK, I get

    $ ./configure --prefix=$HOME/r-svn-test --cache-file=config.cache STRIP=true && make -j8
    ...
    checking for Fortran flag to compile .f95 files... none
    checking for gfortran option to support OpenMP... -fopenmp
    checking for recommended packages... ls: cannot access './src/library/Recommended/MASS_*.tar.gz': No such file or directory
    no
    configure: error: Some of the recommended packages are missing
      Use --without-recommended-packages if this was intentional

However, when I configure --without-recommended-packages and make
install, the build works and both Readline issues are fixed in the
result.

Thank you!

On Tue, May 24, 2016 at 12:47:43PM -0400, Keith O'Hara wrote:
> svn checkout https://svn.r-project.org/R/trunk/ <target-directory>
> 
> > On May 24, 2016, at 12:45 PM, frederik at ofb.net wrote:
> > 
> > I agree with Martin's summary of the situation, and with the updated
> > NEWS entry.
> > 
> > I'm not familiar with Subversion, can you tell me the command to use?
> > 
> > (I tried "svn co https://svn.r-project.org/R/" but it seems to be
> > downloading all branches)
> > 
> > Frederick
> > 
> > On Tue, May 24, 2016 at 04:30:11PM +0200, Martin Maechler wrote:
> >>>>>>> peter dalgaard <pdalgd at gmail.com>
> >>>>>>>    on Tue, 24 May 2016 13:47:27 +0200 writes:
> >> 
> >>> I had a regression in config.site so the nightly build didn't. Retrying....
> >>> Looks like it will build, but the ctl-R, ctl-C bug is still present on OSX (w/Simon's libs). This _was_ fixed for a while, was it not?
> >> 
> >> I thought it was never fixed, for readline versions 5.x (or all
> >> of readline_version < 6.3 ?) because the patch assumed features
> >> not available, e.g., for Frederik (who got compilation errors
> >> which I think you confirmed on pre-6 readline).
> >> 
> >> I remember you having two different readlines installed on OSX
> >> but the standard Mac binary (from CRAN, i.e. Simon) would use
> >> the old readline version ?
> >> 
> >> so that whole  resetReadline() solution is now conditionalized inside
> >> 
> >> #if defined(RL_READLINE_VERSION) && RL_READLINE_VERSION >= 0x0603
> >>   ...
> >>   ...
> >> #endif
> >> 
> >> and hence the previous code (which is buggy) is used for
> >> readline versions < 6.3.
> >> As a consequence the bug is only fixed for readline >= 6.3,
> >> because the current patch did not compile and hence seemed not
> >> appropriate for  readline < 6.3 (and hence the above conditionalization).
> >> 
> >> 
> >>> (The NEWS entry is also wrong: The issue existed before readline 6.3)
> >> 
> >> Aah.. you are right. The API change with 6.3 was for the other, the
> >> "SIGWINCH" bug.
> >> 
> >> Here's a an update proposal for that NEWS entry :
> >> 
> >>         • The API for readline libraries >= 6.3 had changed such
> >>           terminal window resizes where no longer properly signalled
> >>           (PR#16604).  Also, ‘Ctrl C’ in incremental search behaved
> >>           confusingly in R (unix) consoles (PR#16603) also for older
> >>           readline versions. These have been fixed (for readline >=
> >>           6.3 only), thanks to patches by Frederick Eaton.
> >> 
> >> 
> >> Martin
> >> 
> >>> -pd
> >> 
> >>> On 24 May 2016, at 12:55 , Martin Maechler <maechler at stat.math.ethz.ch> wrote:
> >> 
> >>>> 
> >>>> Can you (Frederick, Peter, Keith, but ideally others, too)
> >>>> confirm that you don't see any problems anymore, when building a
> >>>> version of R-devel from sources that are newer 
> >>>> than (or equal to)  svn revision 70632 (2016-05-19 10:59:51, see below)?
> >>>> 
> >>>> I'm asking because the question is open if these should be
> >>>> "back ported" to R 3.3.0 patched or not.
> >>>> 
> >>>> Best regards,
> >>>> Martin
> >>>> 
> >>>>>>>>> Martin Maechler <maechler at stat.math.ethz.ch>
> >>>>>>>>> on Thu, 19 May 2016 11:02:48 +0200 writes:
> >>>> 
> >>>>>>>>> <frederik at ofb.net>
> >>>>>>>>> on Wed, 18 May 2016 15:03:31 -0700 writes:
> >>>> 
> >>>>>>> Readline <= 6.2 shouldn't require the SIGWINCH patch, so
> >>>>>>> if older versions have trouble finding rl_resize_terminal
> >>>>>>> then you could wrap a macro around that part.
> >>>> 
> >>>>>> I find python related patches that use
> >>>> 
> >>>>>> #ifdef HAVE_RL_RESIZE_TERMINAL
> >>>> 
> >>>>>> so they must have configured for that.  We could and
> >>>>>> probably should do the same, but as a Linux_only guy
> >>>>>> currently (even basically only one flavor of Linux), I'd
> >>>>>> appreciate others to produce code for that.
> >>>> 
> >>>>> Actually that was easy (in hindsight.. I took too long!)
> >>>>> enough, so I've now committed
> >>>> 
> >>>>> ------------------------------------------------------------------------
> >>>>> r70632 | maechler | 2016-05-19 10:59:51 +0200 (Thu, 19 May 2016) | 1 line
> >>>>> Changed paths:
> >>>>> M configure
> >>>>> M configure.ac
> >>>>> M src/include/config.h.in
> >>>>> M src/unix/sys-std.c
> >>>> 
> >>>>> check for rl_resize_terminal() now
> >>>>> ------------------------------------------------------------------------
> >>>> 
> >>>>> ... and Keith should even not see the warning anymore
> >>>>> (nor Peter the error, when compiling using readline 5.x instead of 6.[23]).
> >>>> 
> >>>> 
> >>>> [...........]
> >> 
> >>> -- 
> >>> Peter Dalgaard, Professor,
> >>> Center for Statistics, Copenhagen Business School
> >>> Solbjerg Plads 3, 2000 Frederiksberg, Denmark
> >>> Phone: (+45)38153501
> >>> Office: A 4.23
> >>> Email: pd.mes at cbs.dk  Priv: PDalgd at gmail.com
> >> 
>