Re: Documentation patch - Mailing list pgadmin-hackers
From | Akshay Joshi |
---|---|
Subject | Re: Documentation patch |
Date | |
Msg-id | CANxoLDdvCXdS3DnbNG7NrFeXNiKp5An-EEwoJbngBkEF8uRzQg@mail.gmail.com Whole thread Raw |
In response to | Re: Documentation patch (Dave Page <dpage@pgadmin.org>) |
Responses |
Re: Documentation patch
(Dave Page <dpage@pgadmin.org>)
|
List | pgadmin-hackers |
On Wed, Mar 5, 2014 at 3:01 PM, Dave Page <dpage@pgadmin.org> wrote:
On Wed, Mar 5, 2014 at 9:27 AM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:HiI have reviewed the patch. It is working fine on Linux, but on Windows it is not compiling gives "initialization of 'helppath' is skipped by 'case' label" error in misc.cpp (line no 735).HiThanks. I'm not sure if Ads will have a Windows build environment available to him or not. Is it a simple fix?
Yes. Variable 'helppath' is declared inside the case label, so if we declare outside the switch statement then it will get fixed.
On Tue, Mar 4, 2014 at 3:21 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:Sure--On Tue, Mar 4, 2014 at 3:16 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:Akshay,
Can you please take a look at this?--On Tue, Mar 4, 2014 at 3:15 PM, Dave Page <dpage@pgadmin.org> wrote:Ashesh, can you have someone look at this and Ads' other patch please?
Thanks.> --
On Mon, Mar 3, 2014 at 11:00 PM, Andreas 'ads' Scherbaum
<adsmail@wars-nicht.de> wrote:
>
> Hi,
>
> while scrolling through the pgAdmin3 code I found some places where the
> Greenplum links are outdated. The attached patch fixes this.
>
> Note: the current code will update the configuration and write a new link.
> Not sure if this is the best thing to do, or just replace the link on the
> fly.
>
> Also the patch does a better searching for documentation, by looking up all
> possible installed Greenplum versions. Not exactly sure when __WXMSW__ is
> set - this path is not optimized (pgAdmin3.cpp, around line 1417), I just
> added a few updated directories.
>
>
> Regards,
>
> --
> Andreas 'ads' Scherbaum
> German PostgreSQL User Group
> European PostgreSQL User Group - Board of Directors
> Volunteer Regional Contact, Germany - PostgreSQL Project
>
>
> Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
--Thanks & Regards,
Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company
----
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Akshay Joshi
Principal Software Engineer
Phone: +91 20-3058-9517
Mobile: +91 976-788-8246
Mobile: +91 976-788-8246
pgadmin-hackers by date: