Fix for RM2811 - Mailing list pgadmin-hackers

From Harshal Dhumal
Subject Fix for RM2811
Date
Msg-id CAFiP3vycAMc3WJQm7ZBkRg4vMy32oy3jp6Sd+yQTu3sNB0Sv3Q@mail.gmail.com
Whole thread Raw
Responses Re: Fix for RM2811  (Dave Page <dpage@pgadmin.org>)
List pgadmin-hackers
Hi,

Please find attached patch to fix RM2811

Issue was caused due to assumption made when current position in log file while reading it reaches to last line then
we were assuming process is finished. However this is not the case. Background process may be busy performing
some other task and logs might not be logged to file immediately. So we should also check process exit code along
with above condition.

Apart from above this patch also includes minor fix related to status text colour.

--
Harshal Dhumal
Sr. Software Engineer

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

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: [pgAdmin4][Patch]: RM-2859: Can't create new user
Next
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Ensure we don't show the full path on the serverwhen