Re: YA readline 4.2 patch - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: YA readline 4.2 patch
Date
Msg-id 200106110431.f5B4V8k16391@candle.pha.pa.us
Whole thread Raw
In response to YA readline 4.2 patch  (Jason Tishler <Jason.Tishler@dothill.com>)
Responses Re: YA readline 4.2 patch
List pgsql-patches
Your patch has been added to the PostgreSQL unapplied patches list at:

    http://candle.pha.pa.us/cgi-bin/pgpatches

I will try to apply it within the next 48 hours.

> The attached patch enables PostgreSQL CVS to build cleanly under Cygwin
> when built against readline 4.2.  Specifically, it handles the deprecation
> of
>
>     filename_completion_function()
>
> with preference for
>
>     rl_filename_completion_function()
>
> Although, I was motivated by Cygwin support, IMO this patch is appropriate
> for all platforms.  To quote from the readline source:
>
>     #if 0
>     /* Backwards compatibility (compat.c).  These will go away sometime. */
>     ...
>     extern READLINE_EXPORT(char, *filename_completion_function) ...
>     #endif
>
> Note that this patch is modeled after the one by Peter Eisentraut for
> completion_matches():
>
>     http://www.ca.postgresql.org/~petere/readline42.html
>
> I tested this patch under the following environments:
>
>     Cygwin with readline 4.1
>     Cygwin with readline 4.2
>     Linux with readline 2.2.1
>     Linux with readline 4.2
>
> and it behaved as expected.
>
> Thanks,
> Jason
>
> --
> Jason Tishler
> Director, Software Engineering       Phone: 732.264.8770 x235
> Dot Hill Systems Corp.               Fax:   732.264.8798
> 82 Bethany Road, Suite 7             Email: Jason.Tishler@dothill.com
> Hazlet, NJ 07730 USA                 WWW:   http://www.dothill.com

[ Attachment, skipping... ]

>
> ---------------------------(end of broadcast)---------------------------
> TIP 1: subscribe and unsubscribe commands go to majordomo@postgresql.org

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: Cursor support buffer patch
Next
From: Marko Kreen
Date:
Subject: sequence.c compile failure