Re: PATCH: Add libssh2 directory in CPPFLAGS (VPATH issue) - Mailing list pgadmin-hackers

From Dave Page
Subject Re: PATCH: Add libssh2 directory in CPPFLAGS (VPATH issue)
Date
Msg-id CA+OCxozFYf-_5Uh6JT17cMMt78WUBRZwN+h5j0QPvuED9EaoDA@mail.gmail.com
Whole thread Raw
In response to Re: PATCH: Add libssh2 directory in CPPFLAGS (VPATH issue)  (Magnus Hagander <magnus@hagander.net>)
Responses Re: PATCH: Add libssh2 directory in CPPFLAGS (VPATH issue)  (Guillaume Lelarge <guillaume@lelarge.info>)
List pgadmin-hackers
On Mon, May 13, 2013 at 12:25 PM, Magnus Hagander <magnus@hagander.net> wrote:
> On Thu, May 9, 2013 at 1:14 PM, Dave Page <dpage@pgadmin.org> wrote:
>>
>> Hi
>>
>>
>> On Thu, May 9, 2013 at 12:07 PM, Ashesh Vashi
>> <ashesh.vashi@enterprisedb.com> wrote:
>>>
>>> Hi Dave,
>>>
>>> I needed to modify the configuration at quite a lot places to make it
>>> work properly.
>>> And, looks like the pgscript has not been compiled by anyone from long
>>> time and I can say that because, it has been horribly broken.
>>> If we're going to support the stand-alone pgscript application, we will
>>> need to modify the code accordingly.
>>> There are many changes done in the pgadmin3 code, which are shared by
>>> pgscript. And, we've not taken care about pgscript affected areas.
>>
>>
>> Honestly, I don't think we need to keep that there. pgScript was
>> originally written as a GSOC project by Mickael Deloison, and later added to
>> pgAdmin as a second project (iirc, Magnus mentored). We included the command
>> line interpretor just to give it a home, but as it hasn't been touched since
>> it was added and isn't used for anything it should probably go.
>>
>> Any comments Magnus, Mickael?
>
>
> Wel, keeping it anywhere *else* is just going to make the problem even
> worse. So the options are to either fix it, or to throw it out completely.
>
> I'm not against throwing it out completely - as you say, nobody seems to
> have touched it in ages and ages. If we *do* want to keep it, we should make
> it part of the main build so that we detect breakage *right away* and not
> years later - much easier to fix that way.

Thanks. I'm pro-ditching it.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


pgadmin-hackers by date:

Previous
From: Magnus Hagander
Date:
Subject: Re: PATCH: Add libssh2 directory in CPPFLAGS (VPATH issue)
Next
From: Dave Page
Date:
Subject: Remove dialogue "Apply" buttons?