mintty crashes on Windows 7

[Brian Inglis]

On 2022-05-06 08:05, Orgad Shaneh wrote:
> On Fri, May 6, 2022 at 1:49 AM Takashi Yano wrote:
>>> Only bash in msys2 package fails.

>> I identified the difference which causes the issue
>> between bash built from original source and msys2 bash.
>> If --enable-readline and --with-installed-readline is specified
>> to configure, the problem causes even with bash built from original
>> source.
>> Also, removing --with-installed-readline from configure and removing
>> READLINE_LDFLAGS= from make in PKGBUILD of MSYS2 bash package solves
>> the issue.
>> So it seems to be a readline problem, not a bug in bash itself.

> Adding @Johannes Schindelin to the loop.
> Thanks for your investigation so far.
> So how do we proceed, assuming MSYS project does want to use the
> installed readline? I checked the readline PKGBUILD, and it only has
> very basic patches, none of them looks suspicious.
> Is (external) readline not used in cygwin/bash?
> Do you suggest that there's a bug in readline, that your change in the
> runtime uncovered? Or is it the other way around?
> What might we break if we revert the part I referenced earlier in
> fhandler_tty.cc?

Cygwin bash and other packages use the latest readline 8.1 library package:

	$ cygstart https://cygwin.com/packages/summary/readline-src.html

	$ cygcheck -l libreadline7
	/usr/bin/cygreadline7.dll
	/usr/bin/cyghistory7.dll

What readline does MSYS2 bash use - it requires libreadline-devel to 
build, but does not have any readline dependency, nor is libreadline 
required by bash:

	https://packages.msys2.org/package/bash
	https://packages.msys2.org/package/libreadline


-- 
Take care. Thanks, Brian Inglis, Calgary, Alberta, Canada

This email may be disturbing to some readers as it contains
too much technical detail. Reader discretion is advised.
[Data in binary units and prefixes, physical quantities in SI.]