Re: Calling pgScript from the command line

From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: Jeremy Palmer <JPalmer(at)linz(dot)govt(dot)nz>, Guillaume Lelarge <guillaume(at)lelarge(dot)info>, "pgadmin-support(at)postgresql(dot)org" <pgadmin-support(at)postgresql(dot)org>
Subject: Re: Calling pgScript from the command line
Date: 2011-01-17 13:16:33
Message-ID: AANLkTinS8ZipUF6QNktUoV9zw2ZG=ZxGArQwqVkAxDGV@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-support

On Mon, Jan 17, 2011 at 07:46, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
> On Sun, Jan 16, 2011 at 23:02, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>> On Sun, Jan 16, 2011 at 9:50 PM, Jeremy Palmer <JPalmer(at)linz(dot)govt(dot)nz> wrote:
>>> It would be really awesome for DBAs admins to be able to run pgscripts from the command line which have developed in pgadmin. Even better if the programme is distributed with the standard win32 download.
>>>
>>> Would it be possible to revive it and integrate xtra/pgScript into the pgadmin?
>>
>> Well it's already in the source tree, and already integrated into the
>> pgAdmin GUI. All that is needed is a patch to fix the bitrot. As
>> always, patches are welcome.
>
> Seems the first issue at least comes from patch
> 93834d477cd3fb91deacc0946ef845622fd6678d which removed our md5
> support. But there seems to be other issues as well. If you (=Jeremy)
> want to look at this, that'd be great - if not, I can probably look at
> it, but not right now.
>
> (btw, we really should look at making it not compile pgscript twice -
> once for pgadmin and once for the one in xtra/. But that's something
> for another day :D)

Here's a patch that seems to make it work. The issues were:

* md5 no longer available - but also not needed, removed.
* qtIdent() was not available in commandline mode, but needed. This in
turn required db/keywords.c to be linked in.

Comments/objections on patch?

--
 Magnus Hagander
 Me: http://www.hagander.net/
 Work: http://www.redpill-linpro.com/

Attachment Content-Type Size
pgs_cli.patch text/x-patch 2.5 KB

In response to

Responses

Browse pgadmin-support by date

  From Date Subject
Next Message Jeremy Palmer 2011-01-17 21:41:02 Re: Calling pgScript from the command line
Previous Message Magnus Hagander 2011-01-17 06:46:45 Re: Calling pgScript from the command line