GNU bug report logs - #55970
29.0.50; Emacs crashes when dragging the scroll bar with "--with-pgtk" and "--without-toolkit-scroll-bars" configure flags

Previous Next

Full log

Message #47 received at 55970 <at> debbugs.gnu.org (full text, mbox):

From: समीर सिंह Sameer Singh
 <lumarzeli30 <at> gmail.com>
To: Robert Pluim <rpluim <at> gmail.com>
Cc: Po Lu <luangruo <at> yahoo.com>, 55970 <at> debbugs.gnu.org,
 Eli Zaretskii <eliz <at> gnu.org>
Subject: Re: bug#55970: 29.0.50; Emacs crashes when dragging the scroll bar
 with "--with-pgtk" and "--without-toolkit-scroll-bars" configure flags
Date: Wed, 15 Jun 2022 19:31:02 +0530

[Message part 1 (text/plain, inline)]

configure is written in sh, not in C, so you need

diff --git a/configure.ac b/configure.ac
index 5b86e90925..c91b7de322 100644
--- a/configure.ac
+++ b/configure.ac
@@ -3436,7 +3436,7 @@ AC_DEFUN
     AC_DEFINE(USE_TOOLKIT_SCROLL_BARS)
     USE_TOOLKIT_SCROLL_BARS=yes
   fi
-elif test "${window_system}" != "x11" && "${window_system}" != "none"; then
+elif test "${window_system}" != "x11" && test "${window_system}" !=
"none"; then
   AC_MSG_ERROR(Non-toolkit scroll bars are not implemented for your system)
 fi


yes, adding "test" here, correctly produces an error.

On Wed, Jun 15, 2022 at 7:20 PM Robert Pluim <rpluim <at> gmail.com> wrote:

> >>>>> On Wed, 15 Jun 2022 08:59:20 +0800, Po Lu <luangruo <at> yahoo.com> said:
>
>     Po> Eli Zaretskii <eliz <at> gnu.org> writes:
>     >> Thanks for testing, I will let it to Po Lu to decide whether that's
>     >> the proper fix in this case.
>
>     Po> Non-toolkit scroll bars use different kinds of events compared to
>     Po> toolkit scroll bars.  PGTK only implements the latter kind, so
> turning
>     Po> toolkit scroll bars off should trigger this piece of configure.ac:
>
>     Po>   elif test "${window_system}" != "x11" && "${window_system}" !=
> "none"; then
>     Po>     AC_MSG_ERROR(Non-toolkit scroll bars are not implemented for
> your system)
>     Po>   fi
>
>     Po> Perhaps Sameer's configure script is out of date and needs to be
>     Po> regenerated.
>
> configure is written in sh, not in C, so you need
>
> diff --git a/configure.ac b/configure.ac
> index 5b86e90925..c91b7de322 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -3436,7 +3436,7 @@ AC_DEFUN
>      AC_DEFINE(USE_TOOLKIT_SCROLL_BARS)
>      USE_TOOLKIT_SCROLL_BARS=yes
>    fi
> -elif test "${window_system}" != "x11" && "${window_system}" != "none";
> then
> +elif test "${window_system}" != "x11" && test "${window_system}" !=
> "none"; then
>    AC_MSG_ERROR(Non-toolkit scroll bars are not implemented for your
> system)
>  fi
>
>
> (or use -a to avoid repeating the 'test')
>
> Robert
> --
>

[Message part 2 (text/html, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.