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.]

--
Problem reports:      https://cygwin.com/problems.html
FAQ:                  https://cygwin.com/faq/
Documentation:        https://cygwin.com/docs.html
Unsubscribe info:     https://cygwin.com/ml/#unsubscribe-simple

Reply via email to