Latest R-devel build failing on OS X

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