I 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).
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 > >