This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Sync readline/ to version 7.0 alpha
- From: Doug Evans <xdje42 at gmail dot com>
- To: Patrick Palka <patrick at parcs dot ath dot cx>
- Cc: Pedro Alves <palves at redhat dot com>, "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>, Eli Zaretskii <eliz at gnu dot org>, Sergio Durigan Junior <sergiodj at redhat dot com>
- Date: Sat, 25 Jul 2015 12:15:26 -0700
- Subject: Re: [PATCH] Sync readline/ to version 7.0 alpha
- Authentication-results: sourceware.org; auth=none
- References: <1436927724-4059-1-git-send-email-patrick at parcs dot ath dot cx> <55B1FB46 dot 5000106 at redhat dot com> <CA+C-WL8BAJ=LUg59Uu8J8_xDqe4wGsOja5-CYcT_pD9Egd_JPA at mail dot gmail dot com>
On Sat, Jul 25, 2015 at 8:33 AM, Patrick Palka <patrick@parcs.ath.cx> wrote:
> On Fri, Jul 24, 2015 at 4:45 AM, Pedro Alves <palves@redhat.com> wrote:
>> On 07/15/2015 03:35 AM, Patrick Palka wrote:
>>> This patch syncs our upstream copy of readline from version 6.2 to the
>>> latest version, 7.0 alpha (released July 10 2015).
>>>
>>> I essentially copied what was done the last time readline was synced,
>>> when Jan updated to readline 6.2 in 2011:
>>> http://sourceware.org/ml/gdb-patches/2011-05/msg00003.html
>>
>>
>>> Getting this kind of change in at the start of the GDB 7.11 development
>>> cycle will allow us to get a lot of passive testing from developers and
>>> from bleeding-edge users.
>>
>> I agree; if this helps find readline issues that affect GDB before
>> readline is released, it'll be a win, because distros tend to use the
>> system readline instead of the one we bundle.
>>
>>> [And as new versions of readline 7.0 are
>>> released, I will try to sync our local copy promptly.]
>>>
>>
>> Thanks.
>>
>>> Since the patch is too big to send inline, I pushed it to
>>> users/ppalka/readline-7.0-update. Hopefully that is more convenient that
>>> sending it as a compressed attachment.
>>
>> Indeed it is. Thanks.
>>
>>>
>>> readline/ChangeLog.gdb:
>>>
>>> Import readline 7.0 alpha
>>> * configure: Regenerate.
>>> * examples/rlfe/configure: Regenerate.
>>>
>>> gdb/ChangeLog:
>>>
>>> * completer.c (gdb_printable_part): Sync with readline function
>>> it is based off of.
>>>
>>> gdb/testsuite/ChangeLog:
>>>
>>> * gdb.gdb/selftest.exp (test_with_self): Update test to now
>>> expect the GDB inferior to no longer immediately stop after
>>> being resumed with "signal SIGINT".
>>
>> Looks good to me.
>
> Thanks for reviewing. I pushed this patch earlier today and not
> surprisingly the buildbot caught some build failures. The ultimate
> cause of these failures is that readline now builds shared libraries
> by default alongside static libraries. I pushed a patch that disables
> the building of readline's shared libraries by default.
Another build failure:
../../testsuite-random/readline/display.c: In function ârl_redisplayâ:
../../testsuite-random/readline/display.c:1032:12: error: âmb_cur_maxâ
undeclared (first use in this function)
if (mb_cur_max > 1 && rl_byte_oriented == 0)
^
../../testsuite-random/readline/display.c:1032:12: note: each
undeclared identifier is reported only once for each function it
appears in
A lot of uses of mb_cur_max are no longer protected with
#ifdef HANDLE_MULTIBYTE
yet its definition (in rl_redisplay) still is.
I don't know what the right fix is, but we need to get this
fixed asap.