Re: [PATCH 1/1] Cygwin: console: Fix the condition to interrupt select() by SIGWINCH

[Corinna Vinschen]

On Aug 15 14:03, Takashi Yano wrote:
> - Add code so that select() is not interrupted by SIGWINCH if it is
>   ignored (SIG_IGN or SIG_DFL).
> ---
>  winsup/cygwin/select.cc | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/winsup/cygwin/select.cc b/winsup/cygwin/select.cc
> index 9cf892801..4e9256b9f 100644
> --- a/winsup/cygwin/select.cc
> +++ b/winsup/cygwin/select.cc
> @@ -1045,7 +1045,9 @@ peek_console (select_record *me, bool)
>else if (!PeekConsoleInputW (h, &irec, 1, &events_read) || 
> !events_read)
>   break;
>fh->acquire_input_mutex (INFINITE);
> -  if (fhandler_console::input_winch == fh->process_input_message ())
> +  if (fhandler_console::input_winch == fh->process_input_message ()
> +   && global_sigs[SIGWINCH].sa_handler != SIG_IGN
> +   && global_sigs[SIGWINCH].sa_handler != SIG_DFL)
>   {
> set_sig_errno (EINTR);
> fh->release_input_mutex ();
> -- 
> 2.21.0

Do you think a similar workaround would help for the signalfd
problem?


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer


signature.asc
Description: PGP signature

Re: [PATCH 1/1] Cygwin: console: Fix workaround for horizontal tab position

[Corinna Vinschen]

On Aug 15 14:02, Takashi Yano wrote:
> - The workaround commit 33a21904a702191cebf0e81b4deba2dfa10a406c
>   does not work as expected if window size is changed while screen
>   is alternated. Fixed.
> ---
>  winsup/cygwin/fhandler_console.cc | 47 +++
>  1 file changed, 29 insertions(+), 18 deletions(-)

Pushed.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer


signature.asc
Description: PGP signature

Re: [PATCH 1/1] Cygwin: console: Fix the condition to interrupt select() by SIGWINCH

[Corinna Vinschen]

On Aug 15 14:03, Takashi Yano wrote:
> - Add code so that select() is not interrupted by SIGWINCH if it is
>   ignored (SIG_IGN or SIG_DFL).
> ---
>  winsup/cygwin/select.cc | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/winsup/cygwin/select.cc b/winsup/cygwin/select.cc
> index 9cf892801..4e9256b9f 100644
> --- a/winsup/cygwin/select.cc
> +++ b/winsup/cygwin/select.cc
> @@ -1045,7 +1045,9 @@ peek_console (select_record *me, bool)
>else if (!PeekConsoleInputW (h, &irec, 1, &events_read) || 
> !events_read)
>   break;
>fh->acquire_input_mutex (INFINITE);
> -  if (fhandler_console::input_winch == fh->process_input_message ())
> +  if (fhandler_console::input_winch == fh->process_input_message ()
> +   && global_sigs[SIGWINCH].sa_handler != SIG_IGN
> +   && global_sigs[SIGWINCH].sa_handler != SIG_DFL)
>   {
> set_sig_errno (EINTR);
> fh->release_input_mutex ();
> -- 
> 2.21.0

Pushed.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer


signature.asc
Description: PGP signature

Re: [PATCH] Cygwin: ldd: Try harder to get dll names

[Corinna Vinschen]

On Aug 14 22:59, Mark Geisert wrote:
> Borrow a trick from strace to lessen occurrences of "??? => ???" in ldd
> output.  Specifically, if the module name isn't found in the usual place
> in the mapped image, use the file handle we have to look up the name.
> 
> ---
>  winsup/utils/Makefile.in |  2 +-
>  winsup/utils/ldd.cc  | 44 +++-
>  2 files changed, 44 insertions(+), 2 deletions(-)

Great idea, pushed.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer


signature.asc
Description: PGP signature