public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
From: Eric Blake <eblake@redhat.com>
To: cygwin@cygwin.com, Steven Penny <svnpenn@gmail.com>
Subject: Re: [ANNOUNCEMENT] Updated: libreadline7-7.0.3-3
Date: Thu, 27 Jul 2017 21:37:00 -0000	[thread overview]
Message-ID: <5dbbf0e4-6374-a9bb-21e5-dd5537e0e19a@redhat.com> (raw)
In-Reply-To: <58f1a28e.0a2c9d0a.f2ec.8160@mx.google.com>


[-- Attachment #1.1: Type: text/plain, Size: 2944 bytes --]


On 04/14/2017 11:33 PM, Steven Penny wrote:
> On Thu, 13 Apr 2017 13:48:04, Eric Blake wrote:
>> Is it still a problem with pselect, where rebuilding with the same
>> configuration as 7.0.1-2 fixes things?

I've got some time today to look at building readline, but for the life
of me, I can't figure out what I'm supposed to be debugging.  You have
so many emails saying "see this earlier URL" that I am lost in what you
are saying is wrong or how to reproduce it.

I'm currently testing with:

bash 4.4.12-3
cygwin 2.8.2-1
libreadline7 7.0.3-3 (or self-built)

> I'm really not sure how to even
>> go about debugging this one, and it's not my highest priority at the
>> moment (I've got coreutils 8.27 to build for cygwin, and autoconf 2.70
>> to release upstream).  So any help is welcome.
> 
> Ok. I have not gone through the whole commit, as it is huge:
> 
> http://cygwin.com/ml/cygwin/2017-01/msg00204.html
> 
> but I did find something. Using:
> 
>    git checkout readline-7.0-alpha~1
> 
> for the last good commit and:
> 
>    git checkout readline-7.0-alpha
> 
> for the first bad commit, I found that the change to the "rl_insert"
> function in
> "text.c" breaks pasting and Alt codes with "chcp.com 65001". Can you
> work with
> this?

Thanks again for trying to narrow things down.  I have recompiled
readline locally with optimizations turned off (so it's easier for me to
see what's going on), and am set up to run gdb on bash with a given
readline executable installed.  If you have really narrowed the problem
to rl_insert(), that's at least something I can investigate.

But where I'm stuck now is what works for you and what you think is
wrong.  Is this something where I can start bash under mintty, or do I
have to start under cmd?  Right there, I already see a difference with
the two environments.  Starting from cmd, I did:

c:\cygwin\bin> od -tx1
<Alt-num2-num3-num4><Enter><Ctrl-d>

which displayed
Ω
00000000 ce a9 0z
0000003

so I did indeed insert GREEK CAPITAL LETTER OMEGA U+03A9.

But trying the same thing under a bash session in minty shows:

ê
0000000 c3 aa 0a
0000003

so that is not the same character.  I'm not sure if a code page change
is supposed to alter what I see.

So I'm back to cmd to try and debug things.  Next, I tried:

c:\cygwin\bin> .\dash
<alt-num2-num3-num4>

and again got Ω; pressing <enter> complains that ./dash: 1: Ω: not found

However, when I try:

c:\cygwin\bin> .\bash --norc
<alt-num2-num3-num4>

the display shows :\251

and hitting <Enter> wipes out that display without doing anything.  So I
_think_ I'm running into the problem you're describing, but want to make
sure, since it is different based on whether I started bash from cmd or
from mintty.

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 619 bytes --]

  parent reply	other threads:[~2017-07-27 17:09 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-02-13 19:53 [ANNOUNCEMENT] Updated: libreadline7-7.0.3-3, libreadline-devel-7.0.3-3 Eric Blake (cygwin)
2017-03-25  0:46 ` Steven Penny
2017-04-13 19:07   ` [ANNOUNCEMENT] Updated: libreadline7-7.0.3-3 Steven Penny
2017-04-14  8:27     ` Eric Blake
2017-04-15 10:59       ` Steven Penny
2017-05-15 18:21         ` Eric Blake
2017-05-15 20:46           ` Chet Ramey
2017-06-18 15:53             ` Steven Penny
2017-07-04 21:53             ` Steven Penny
2017-07-27 21:37         ` Eric Blake [this message]
2017-07-27 21:39           ` Eric Blake
2017-07-27 21:46           ` Steven Penny
2017-07-28  8:28             ` Eric Blake
2017-07-28 14:55               ` Steven Penny
2017-07-28 18:31                 ` Eric Blake
2017-07-28 18:39                   ` Steven Penny
2017-07-28 23:55                     ` Eric Blake
2017-07-29  1:55                       ` Cygwin.bat (was: [ANNOUNCEMENT] Updated: libreadline7-7.0.3-3) Achim Gratz
2017-07-29  2:48                       ` [ANNOUNCEMENT] Updated: libreadline7-7.0.3-3 Doug Henderson
2017-07-29  4:23                         ` Steven Penny
2017-07-30 18:38                           ` Doug Henderson
2017-07-30 19:54                             ` Steven Penny
2017-07-29  8:45                       ` Steven Penny
2017-07-29 10:08                         ` Eric Blake
2017-07-31 18:36                     ` Corinna Vinschen
2017-07-31 20:01                       ` Steven Penny
2017-07-31 20:05                       ` David Macek
2017-07-31 21:13                         ` David Macek
     [not found]                         ` <20170731200146.GD18950@calimero.vinschen.de>
     [not found]                           ` <20170731211327.GG18950@calimero.vinschen.de>
2017-08-01  0:56                             ` Steven Penny
2017-08-01  8:45                               ` Corinna Vinschen
2017-08-01 14:48                                 ` Corinna Vinschen
2017-08-01 18:20                                   ` Steven Penny
2017-08-01 18:54                                     ` Achim Gratz
2017-08-01 19:02                                       ` Eric Blake
2017-08-01  7:22                             ` David Macek
2017-08-01  8:46                               ` Corinna Vinschen
     [not found]                           ` <160b3569-d448-1898-3dcd-b7133a772527@SystematicSw.ab.ca>
2017-08-01  8:44                             ` Corinna Vinschen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5dbbf0e4-6374-a9bb-21e5-dd5537e0e19a@redhat.com \
    --to=eblake@redhat.com \
    --cc=cygwin@cygwin.com \
    --cc=svnpenn@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).