Re: Patch 8.2.4560

2022-03-13 Fir de Conversatie Dominique Pellé
Bram Moolenaar wrote:

> Patch 8.2.4560
> Problem:Suspending with CTRL-Z does not work on DragonFlyBSD.
> Solution:   Adjust #ifdef. (Ozaki Kiichi, closes #9943)
> Files:  src/os_unix.c
...snip...
> ! #if !defined(__ANDROID__) && !defined(__OpenBSD__) && 
> !defined(__DragonFly__)
> ! // This is not required on all systems.  On some systems (at least 
> Android,
> ! // OpenBSD, and DragonFlyBSD) this breaks suspending with CTRL-Z.
>   signal(SIGTSTP, (RETSIGTYPE (*)())sig_tstp);
>   #endif

I wonder whether there is a better way than adding
ifdef values whenever someone reports an issue.

I can't verify but perhaps we also need to check  __FreeBSD__
__NetBSD__, __bsdi__, others ?

It would be nice if someone with such a system could
confirm whether CTRL-Z works or not and whether
adding extra ifdef cases fixes it.

Regards
Dominique

-- 
-- 
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.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/vim_dev/CAON-T_hmrLg%3Ddj5RXrubbTHPnVsEiwNEWxww_uBvC2k4TZafTA%40mail.gmail.com.


Patch 8.2.4560

2022-03-13 Fir de Conversatie Bram Moolenaar


Patch 8.2.4560
Problem:Suspending with CTRL-Z does not work on DragonFlyBSD.
Solution:   Adjust #ifdef. (Ozaki Kiichi, closes #9943)
Files:  src/os_unix.c


*** ../vim-8.2.4559/src/os_unix.c   2022-03-09 14:32:59.617641710 +
--- src/os_unix.c   2022-03-13 17:22:29.407382895 +
***
*** 887,895 
  else
got_tstp = TRUE;
  
! #if !defined(__ANDROID__) && !defined(__OpenBSD__)
! // This is not required on all systems.  On some systems (at least Android
! // and OpenBSD) this breaks suspending with CTRL-Z.
  signal(SIGTSTP, (RETSIGTYPE (*)())sig_tstp);
  #endif
  SIGRETURN;
--- 887,895 
  else
got_tstp = TRUE;
  
! #if !defined(__ANDROID__) && !defined(__OpenBSD__) && !defined(__DragonFly__)
! // This is not required on all systems.  On some systems (at least 
Android,
! // OpenBSD, and DragonFlyBSD) this breaks suspending with CTRL-Z.
  signal(SIGTSTP, (RETSIGTYPE (*)())sig_tstp);
  #endif
  SIGRETURN;
*** ../vim-8.2.4559/src/version.c   2022-03-13 15:52:34.173429915 +
--- src/version.c   2022-03-13 17:25:14.094960245 +
***
*** 752,753 
--- 752,755 
  {   /* Add new patch number below this line */
+ /**/
+ 4560,
  /**/

-- 
Emacs is a nice OS - but it lacks a good text editor.
That's why I am using Vim.  --Anonymous

 /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
///  \\\
\\\sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
 \\\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.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/vim_dev/20220313172837.389F21C79E4%40moolenaar.net.