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: Patrick Palka <patrick at parcs dot ath dot cx>
- To: Doug Evans <xdje42 at gmail dot com>
- 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 15:28:40 -0400
- 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> <CAP9bCMQpzsaGuf5Ym9F3bXqiNtGfvPDdGJ=BpttUFj_HW2y0Uw at mail dot gmail dot com>
On Sat, Jul 25, 2015 at 3:15 PM, Doug Evans <xdje42@gmail.com> wrote:
> 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.
Hmm, I'll just revert this patch for now since I also noticed that I
failed to reapply some local changes to readline's configure.in that
were hid by the rename to configure.ac. Also, the buildbots are
showing new regressions for e.g. AIX. I won't have enough time over
the next few days to manage the fallout of this change.