pgsql ALTER command tab completion

Started by Ross J. Reedstromover 23 years ago4 messagespatches
Jump to latest
#1Ross J. Reedstrom
reedstrm@rice.edu

Here's the patch I mentioned on HACKERS to fix up ALTER command tab
completion. Note that it's based on 7.3 tarball, not CVS HEAD, or 7.3rel
branch. Damn, looking at CVS, this will patch into 7.3rel (just tested,
it does) probably collide with Rod Taylor's patch adding ALTER TRIGGER
stuff. O.K, second patch attached against HEAD - not tested, hand merged.

Ross

--
Ross Reedstrom, Ph.D. reedstrm@rice.edu
Research Scientist phone: 713-348-6166
The Connexions Project http://cnx./rice.edu fax: 713-348-6182
Rice University MS-39
Houston, TX 77005

Attachments:

tab-complete-HEAD.difftext/plain; charset=us-asciiDownload+23-7
tab-complete.difftext/plain; charset=us-asciiDownload+23-8
#2Bruce Momjian
bruce@momjian.us
In reply to: Ross J. Reedstrom (#1)
Re: pgsql ALTER command tab completion

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

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

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

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

Ross J. Reedstrom wrote:

Here's the patch I mentioned on HACKERS to fix up ALTER command tab
completion. Note that it's based on 7.3 tarball, not CVS HEAD, or 7.3rel
branch. Damn, looking at CVS, this will patch into 7.3rel (just tested,
it does) probably collide with Rod Taylor's patch adding ALTER TRIGGER
stuff. O.K, second patch attached against HEAD - not tested, hand merged.

Ross

--
Ross Reedstrom, Ph.D. reedstrm@rice.edu
Research Scientist phone: 713-348-6166
The Connexions Project http://cnx./rice.edu fax: 713-348-6182
Rice University MS-39
Houston, TX 77005

[ Attachment, skipping... ]

[ 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) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073
#3Bruce Momjian
bruce@momjian.us
In reply to: Ross J. Reedstrom (#1)
Re: pgsql ALTER command tab completion

Patch applied. Your HEAD version didn't apply cleanly, so I manually
applied your first version. I made some coding cleanups at the same
time, so please check CVS and make sure it is behaving they way it
should. The manual patch application was quite easy.

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

Ross J. Reedstrom wrote:

Here's the patch I mentioned on HACKERS to fix up ALTER command tab
completion. Note that it's based on 7.3 tarball, not CVS HEAD, or 7.3rel
branch. Damn, looking at CVS, this will patch into 7.3rel (just tested,
it does) probably collide with Rod Taylor's patch adding ALTER TRIGGER
stuff. O.K, second patch attached against HEAD - not tested, hand merged.

Ross

--
Ross Reedstrom, Ph.D. reedstrm@rice.edu
Research Scientist phone: 713-348-6166
The Connexions Project http://cnx./rice.edu fax: 713-348-6182
Rice University MS-39
Houston, TX 77005

[ Attachment, skipping... ]

[ 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) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073
#4Ross J. Reedstrom
reedstrm@rice.edu
In reply to: Bruce Momjian (#3)
Re: pgsql ALTER command tab completion

Patch applied. Your HEAD version didn't apply cleanly, so I manually
applied your first version. I made some coding cleanups at the same
time, so please check CVS and make sure it is behaving they way it
should. The manual patch application was quite easy.

I'll check it. Thanks.

Ross