Re: pgAdmin 4 commit: Sporadically crashes on Windows when exit. Fixes #317

From: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
To: Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: pgAdmin 4 commit: Sporadically crashes on Windows when exit. Fixes #317
Date: 2018-03-15 10:44:17
Message-ID: CANxoLDeJK05HLZfxC7GanritfE9Kao9eM71_bW9q2yY_OsMaDg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Thanks patch applied

On Thu, Mar 15, 2018 at 4:03 PM, Murtuza Zabuawala <
murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:

> Hi Akshay,
>
> Given commit introduce PEP8 issue, PFA minor patch to fix the issue.
>
> --
> Regards,
> Murtuza Zabuawala
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
>
> On Thu, Mar 15, 2018 at 1:27 PM, Akshay Joshi <
> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>
>> Sporadically crashes on Windows when exit. Fixes #3177
>> 1) Shutdown the python server properly.
>> 2) Disabled "Shutdown server" menu till server is not successfully
>> started.
>>
>> Initial patch sent by Maxim, modified by Akshay Joshi.
>>
>> Branch
>> ------
>> master
>>
>> Details
>> -------
>> https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdif
>> f;h=54b1a79cb6f90ac9b26db57e616dc750fb2c6b54
>> Author: Maxim Zakharov <dp(dot)maxime(at)gmail(dot)com>
>>
>> Modified Files
>> --------------
>> runtime/Server.cpp | 11 +++++++++++
>> runtime/Server.h | 3 +++
>> runtime/TrayIcon.cpp | 11 +++++++++++
>> runtime/TrayIcon.h | 4 ++++
>> runtime/pgAdmin4.cpp | 46 ++++++++++++++++++++++++++++++
>> ++++++++++++++
>> runtime/pgAdmin4.h | 1 +
>> web/pgadmin/misc/__init__.py | 16 ++++++++++++++-
>> 7 files changed, 91 insertions(+), 1 deletion(-)
>>
>>
>

--
*Akshay Joshi*

*Sr. Software Architect *

*Phone: +91 20-3058-9517Mobile: +91 976-788-8246*

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2018-03-15 10:53:36 pgAdmin 4 commit: Exit CI run if the PEP-8 checks fail.
Previous Message Akshay Joshi 2018-03-15 10:43:18 pgAdmin 4 commit: Fixed PEP8 issue