This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] [RFC] Sync readline to version 6.3 patchlevel 8
- From: Pedro Alves <palves at redhat dot com>
- To: Patrick Palka <patrick at parcs dot ath dot cx>, gdb-patches at sourceware dot org
- Cc: jan dot kratochvil at redhat dot com
- Date: Thu, 14 May 2015 10:29:19 +0100
- Subject: Re: [PATCH] [RFC] Sync readline to version 6.3 patchlevel 8
- Authentication-results: sourceware.org; auth=none
- References: <1431562331-20448-1-git-send-email-patrick at parcs dot ath dot cx>
On 05/14/2015 01:12 AM, Patrick Palka wrote:
> This patch syncs our upstream copy of readline to version 6.3
> patchlevel 8.
>
> I basically copied what was done when Jan updated to readline 6.2 in
> 2011: http://sourceware.org/ml/gdb-patches/2011-05/msg00003.html
>
> Specifically, I:
>
> 1. Extracted the readline 6.3 tarball on top of readline/
> 2. Applied patches 1-8 from ftp://ftp.cwru.edu/pub/bash/readline-6.3-patches/
> 3. Omitted all the files in doc/ that were intentionally omitted before.
> 4. Regenerated readline/configure and readline/examples/rlfe/configure
> using autoconf 2.64. No other configure files needed regenerating.
> 5. Reapplied the only local patch since the update to readline 6.2 that
> has not already been applied to readline 6.3, 05942d8a1 ("Fix
> executable indicator in file name completeion on Windows."). This
> particular patch has been applied upstream but readline 6.3 does not
> have it. Whether or not a local patch has already been applied to
> readline 6.3 was determined via manual inspection. (Wasn't too bad
> really.)
>
> The new files to make it into the tree are:
>
> colors.{c,h}
> configure.ac
> parse-colors.{c,h}
> examples/hist_erasedups.c
> examples/hist_purgecmd.c
>
> Deleted files:
>
> configure.in
>
> I've been using this patch locally for a few months now and I've
> experienced only a single regression which has already been preemptively
> fixed by 8900d71e3 ("Explicitly call rl_resize_terminal() in TUI's
> SIGWINCH handler"). Other than that, no issues in either the CLI or the
> TUI, or changes in the testsuite. Though I have only been able to test
> this patch on Linux.
>
> Have I missed anything? What else should be done in order to sync our
> copy of readline? Unlike the last sync from 5.1 to 6.2, this sync to
> 6.3 is quite minor and hopefully painless.
>
> Because the commit is so big and so mechanical I elected to create a
> branch users/ppalka/readline-6.3-update where you can view/test the
> change.
(I haven't looked at the branch,) but it all sounds great to me.
Many thanks for doing all this.
Thanks,
Pedro Alves