Unix Technical Forum

Small bug fix in readline promt handling

This is a discussion on Small bug fix in readline promt handling within the pgsql Bugs forums, part of the PostgreSQL category; --> Hi! When using %[ and %] (prompt length ignore area) in .psqlrc's prompt formats, psql sends a \001 before ...


Go Back   Unix Technical Forum > Database Server Software > PostgreSQL > pgsql Bugs

Register FAQ Members List Calendar Search Today's Posts Mark Forums Read
  #1 (permalink)  
Old 04-10-2008, 09:35 AM
Martin Pitt
 
Posts: n/a
Default Small bug fix in readline promt handling

Hi!

When using %[ and %] (prompt length ignore area) in .psqlrc's prompt
formats, psql sends a \001 before the actual readline commands
(RL_PROMPT_{START,END}_IGNORE). This is the way bash handles the
shell's PSx format, but it is not a readline command, but a bash
specific quirk.

Example .psqlrc:
\set PROMPT1 '%[%033[0;35m%]%/@%m%R%#%[%033[0;22m%] '

-> not correctly handled, the cursor will be in the middle of the
prompt.

Attached patch fixes this. This was investigated and patched by Aaron
Schrab <aaron@schrab.com>, full details at http://bugs.debian.org/343616

Thank you for considering this, and have a merry christmas!

Martin

--
Martin Pitt http://www.piware.de
Ubuntu Developer http://www.ubuntu.com
Debian Developer http://www.debian.org

In a world without walls and fences, who needs Windows and Gates?

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2 (GNU/Linux)

iD8DBQFDqu9HDecnbV4Fd/IRAsJ9AJ9paQ2QA28gYJ2n302vQGDB8MvewACg4VAG
z6WAEAb9x7QMzMlDwEwSnk4=
=gaq4
-----END PGP SIGNATURE-----

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #2 (permalink)  
Old 04-10-2008, 09:36 AM
Bruce Momjian
 
Posts: n/a
Default Re: Small bug fix in readline promt handling


Your patch has been added to the PostgreSQL unapplied patches list at:

http://momjian.postgresql.org/cgi-bin/pgpatches

It will be applied as soon as one of the PostgreSQL committers reviews
and approves it.

---------------------------------------------------------------------------


Martin Pitt wrote:
-- Start of PGP signed section.
> Hi!
>
> When using %[ and %] (prompt length ignore area) in .psqlrc's prompt
> formats, psql sends a \001 before the actual readline commands
> (RL_PROMPT_{START,END}_IGNORE). This is the way bash handles the
> shell's PSx format, but it is not a readline command, but a bash
> specific quirk.
>
> Example .psqlrc:
> \set PROMPT1 '%[%033[0;35m%]%/@%m%R%#%[%033[0;22m%] '
>
> -> not correctly handled, the cursor will be in the middle of the
> prompt.
>
> Attached patch fixes this. This was investigated and patched by Aaron
> Schrab <aaron@schrab.com>, full details at http://bugs.debian.org/343616
>
> Thank you for considering this, and have a merry christmas!
>
> Martin
>
> --
> Martin Pitt http://www.piware.de
> Ubuntu Developer http://www.ubuntu.com
> Debian Developer http://www.debian.org
>
> In a world without walls and fences, who needs Windows and Gates?


[ Attachment, skipping... ]
-- End of PGP section, PGP failed!

--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

---------------------------(end of broadcast)---------------------------
TIP 5: don't forget to increase your free space map settings

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #3 (permalink)  
Old 04-10-2008, 09:36 AM
Tom Lane
 
Posts: n/a
Default Re: Small bug fix in readline promt handling

Martin Pitt <martin@piware.de> writes:
> When using %[ and %] (prompt length ignore area) in .psqlrc's prompt
> formats, psql sends a \001 before the actual readline commands
> (RL_PROMPT_{START,END}_IGNORE). This is the way bash handles the
> shell's PSx format, but it is not a readline command, but a bash
> specific quirk.


Fixed back through 8.0 (we don't seem to have that code in 7.x branches).
Thanks for the report.

regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 4: Have you searched our list archives?

http://archives.postgresql.org

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
Reply


Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off
Trackbacks are On
Pingbacks are On
Refbacks are On
Forum Jump


All times are GMT. The time now is 02:30 AM.


Powered by vBulletin® Version 3.6.5
Copyright ©2000 - 2008, Jelsoft Enterprises Ltd.
SEO by vBSEO 3.2.0
www.UnixAdminTalk.com