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 >
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 >>
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
> >>
>
>>>>> Keith O'Hara <keith.ohara at nyu.edu> >>>>> on Tue, 24 May 2016 12:47:43 -0400 writes:> svn checkout https://svn.r-project.org/R/trunk/ <target-directory> yes, indeed. thank you, Keith. and from then on only cd <target-directory> svn up (which is short for 'svn update'). Another hint: Then do *not* build in the source directory but in what we called a "build directory"; i.e., something like (from scratch; including the only-once needed "checkout") : svn checkout https://svn.r-project.org/R/trunk/ R cd R tools/rsync-recommended mkdir ../build-R cd ../build-R ../R/configure make make check and I then never run 'make install', but rather use symbolic link from ..../build-R/bin/R to something like ~/bin/R-devel i.e., cd ~/bin ln -s ..../build-R/bin/R R-devel Martin >> 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 >>>