Re: Patch 8.0.1071

[Christian Brabandt]

On Do, 07 Sep 2017, Bram Moolenaar wrote:

> Patch 8.0.1071
> Problem:$TERM names starting with "putty" and "cygwin" are likely to have
> a dark background, but are not recognized.
> Solution:   Only check the first few characters of $TERM to match "putty" or
> "cygwin". (Christian Brabandt)
> Files:  src/option.c

Oh wow that was fast. Thanks.

Best,
Christian
-- 
Wir alle leben vom Vergangnen und gehen am Vergangenen zugrunde.
-- Goethe, Maximen und Reflektionen, Nr. 308

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Patch 8.0.1071

[Bram Moolenaar]

Patch 8.0.1071
Problem:$TERM names starting with "putty" and "cygwin" are likely to have
a dark background, but are not recognized.
Solution:   Only check the first few characters of $TERM to match "putty" or
"cygwin". (Christian Brabandt)
Files:  src/option.c


*** ../vim-8.0.1070/src/option.c2017-09-02 18:33:52.453554469 +0200
--- src/option.c2017-09-07 22:37:23.295474873 +0200
***
*** 4024,4031 
   * This is just guessing!  Recognized are:
   * "linux"Linux console
   * "screen.linux"   Linux console with screen
!  * "cygwin"   Cygwin shell
!  * "putty"Putty program
   * We also check the COLORFGBG environment variable, which is set by
   * rxvt and derivatives. This variable contains either two or three
   * values separated by semicolons; we want the last value in either
--- 4024,4031 
   * This is just guessing!  Recognized are:
   * "linux"Linux console
   * "screen.linux"   Linux console with screen
!  * "cygwin.*" Cygwin shell
!  * "putty.*"  Putty program
   * We also check the COLORFGBG environment variable, which is set by
   * rxvt and derivatives. This variable contains either two or three
   * values separated by semicolons; we want the last value in either
***
*** 4035,4049 
  term_bg_default(void)
  {
  #if defined(WIN3264)
! /* DOS console nearly always black */
  return (char_u *)"dark";
  #else
  char_u*p;
  
  if (STRCMP(T_NAME, "linux") == 0
|| STRCMP(T_NAME, "screen.linux") == 0
!   || STRCMP(T_NAME, "cygwin") == 0
!   || STRCMP(T_NAME, "putty") == 0
|| ((p = mch_getenv((char_u *)"COLORFGBG")) != NULL
&& (p = vim_strrchr(p, ';')) != NULL
&& ((p[1] >= '0' && p[1] <= '6') || p[1] == '8')
--- 4035,4049 
  term_bg_default(void)
  {
  #if defined(WIN3264)
! /* DOS console is nearly always black */
  return (char_u *)"dark";
  #else
  char_u*p;
  
  if (STRCMP(T_NAME, "linux") == 0
|| STRCMP(T_NAME, "screen.linux") == 0
!   || STRNCMP(T_NAME, "cygwin", 6) == 0
!   || STRNCMP(T_NAME, "putty", 5) == 0
|| ((p = mch_getenv((char_u *)"COLORFGBG")) != NULL
&& (p = vim_strrchr(p, ';')) != NULL
&& ((p[1] >= '0' && p[1] <= '6') || p[1] == '8')
*** ../vim-8.0.1070/src/version.c   2017-09-07 22:24:37.372486913 +0200
--- src/version.c   2017-09-07 22:36:04.779993764 +0200
***
*** 771,772 
--- 771,774 
  {   /* Add new patch number below this line */
+ /**/
+ 1071,
  /**/

-- 
>From "know your smileys":
 (:-#   Said something he shouldn't have

 /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
///sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\  an exciting new programming language -- http://www.Zimbu.org///
 \\\help me help AIDS victims -- http://ICCF-Holland.org///

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Patch 8.0.1070

[Bram Moolenaar]

Patch 8.0.1070
Problem:Terminal test is flaky on Mac.
Solution:   Add Test_terminal_noblock() to list of flaky tests.
Files:  src/testdir/runtest.vim


*** ../vim-8.0.1069/src/testdir/runtest.vim 2017-07-30 21:37:54.631396197 
+0200
--- src/testdir/runtest.vim 2017-09-07 22:23:19.321000898 +0200
***
*** 233,238 
--- 233,239 
\ 'Test_quoteplus()',
\ 'Test_quotestar()',
\ 'Test_reltime()',
+   \ 'Test_terminal_noblock()',
\ 'Test_with_partial_callback()',
\ ]
  
*** ../vim-8.0.1069/src/version.c   2017-09-07 20:18:35.430209988 +0200
--- src/version.c   2017-09-07 22:24:01.588722327 +0200
***
*** 771,772 
--- 771,774 
  {   /* Add new patch number below this line */
+ /**/
+ 1070,
  /**/

-- 
>From "know your smileys":
 :q vi user saying, "How do I get out of this damn emacs editor?"

 /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
///sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\  an exciting new programming language -- http://www.Zimbu.org///
 \\\help me help AIDS victims -- http://ICCF-Holland.org///

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Patch 8.0.1069

[Bram Moolenaar]

Patch 8.0.1069
Problem:Still get CTRL-X sometimes for t_RS request.
Solution:   Also skip 0x18 after a key code response.
Files:  src/term.c


*** ../vim-8.0.1068/src/term.c  2017-09-07 12:59:18.348291737 +0200
--- src/term.c  2017-09-07 20:18:11.234367459 +0200
***
*** 4741,4749 
key_name[0] = (int)KS_EXTRA;
key_name[1] = (int)KE_IGNORE;
slen = i + 1 + (tp[i] == ESC);
!   if (tp[i] == 0x07 && i + 1 < len && tp[i + 1] == 0x18)
!   /* Sometimes the 0x07 is followed by 0x18, unclear
!* when this happens. */
++slen;
  # ifdef FEAT_EVAL
set_vim_var_string(VV_TERMRGBRESP, tp, slen);
--- 4741,4750 
key_name[0] = (int)KS_EXTRA;
key_name[1] = (int)KE_IGNORE;
slen = i + 1 + (tp[i] == ESC);
!   if (rcs_status == STATUS_SENT
!&& slen < len && tp[slen] == 0x18)
!   /* Some older xterm send 0x18 for the T_RS request,
!* skip it here. */
++slen;
  # ifdef FEAT_EVAL
set_vim_var_string(VV_TERMRGBRESP, tp, slen);
***
*** 4793,4798 
--- 4794,4804 
key_name[0] = (int)KS_EXTRA;
key_name[1] = (int)KE_IGNORE;
slen = i + 1 + (tp[i] == ESC);
+   if (rcs_status == STATUS_SENT
+&& slen < len && tp[slen] == 0x18)
+   /* Some older xterm send 0x18 for the T_RS request,
+* skip it here. */
+   ++slen;
break;
}
  }
*** ../vim-8.0.1068/src/version.c   2017-09-07 12:59:18.352291712 +0200
--- src/version.c   2017-09-07 20:17:47.11452 +0200
***
*** 771,772 
--- 771,774 
  {   /* Add new patch number below this line */
+ /**/
+ 1069,
  /**/

-- 
>From "know your smileys":
 O:-)   Saint

 /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
///sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\  an exciting new programming language -- http://www.Zimbu.org///
 \\\help me help AIDS victims -- http://ICCF-Holland.org///

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Test failure on linux ppc

[Christian Brabandt]

On Do, 07 Sep 2017, Elimar Riesebieter wrote:

> * Christian Brabandt  [2017-09-06 22:14 +0200]:
> 
> > 
> [...]
> > Did this work with a previous version? If you run the ppc compiled vim 
> > and do :e /.config/git/config  what is the filetype (:set filetype?)
> 
> Permission Denied

Hm, looks like this is not easily catchable...


Best,
Christian
-- 
Im Leben ist das Böse oft preiswerter als das Dumme.
-- Mathias Richling

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Patch 8.0.1068

[Bram Moolenaar]

Patch 8.0.1068 (after 8.0.1066)
Problem:Vandyke SecureCRT terminal can't handle cursor mode request.
(Steven Hartland)
Solution:   Fix pointer computation. (closes #2008)
Files:  src/term.c


*** ../vim-9.0.1067/src/term.c  2017-09-06 22:08:11.829623019 +0200
--- src/term.c  2017-09-07 12:32:32.842566672 +0200
***
*** 4583,4589 
/* PuTTY sends 0;136;0
 * vandyke SecureCRT sends 1;136;0 */
if (version == 136
!   && STRNCMP(tp + extra - 3, ";136;0c", 8) == 0)
is_not_xterm = TRUE;
  
/* Konsole sends 0;115;0 */
--- 4583,4589 
/* PuTTY sends 0;136;0
 * vandyke SecureCRT sends 1;136;0 */
if (version == 136
!   && STRNCMP(tp + extra - 1, ";136;0c", 7) == 0)
is_not_xterm = TRUE;
  
/* Konsole sends 0;115;0 */
*** ../vim-8.0.1067/src/version.c   2017-09-06 23:40:05.517366267 +0200
--- src/version.c   2017-09-07 12:28:03.444279389 +0200
***
*** 771,772 
--- 771,774 
  {   /* Add new patch number below this line */
+ /**/
+ 1068,
  /**/

-- 
>From "know your smileys":
 ~#:-(  I just washed my hair, and I can't do nuthin' with it.

 /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
///sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\  an exciting new programming language -- http://www.Zimbu.org///
 \\\help me help AIDS victims -- http://ICCF-Holland.org///

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Test failure on linux ppc

[Elimar Riesebieter]

* Christian Brabandt  [2017-09-06 22:14 +0200]:

> 
[...]
> Did this work with a previous version? If you run the ppc compiled vim 
> and do :e /.config/git/config  what is the filetype (:set filetype?)

Permission Denied

> What for the other upstart paths? You probably need a :filetype on

* Bram Moolenaar  [2017-09-06 21:44 +0200]:

[...]
> It works fine for most users.  What is special about this system?
> The common part seems to be the ".config" directory.

There was a leftover from an old ALSA bug. I deleted the directory
/.config owned by root, created in 2013 and everything works as
expected now.

BTW the former version 8.0.0946 compiled fine with /.config ...

Elimar
-- 
  Excellent day for drinking heavily.
  Spike the office water cooler;-)

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.