Re: [pgAdmin][RM-2341]: Add menu option for starting PSQL

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Nikhil Mohite <nikhil(dot)mohite(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
Subject: Re: [pgAdmin][RM-2341]: Add menu option for starting PSQL
Date: 2021-06-10 14:50:53
Message-ID: CA+OCxoyAd7TUNjs6xyUrDR32YKNhfJTzzzH4JP=StKw0PoVXVQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Thu, Jun 10, 2021 at 2:22 PM Nikhil Mohite <
nikhil(dot)mohite(at)enterprisedb(dot)com> wrote:

> Hi Dave,
>
> On Thu, Jun 10, 2021 at 6:37 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Hi
>>
>> On Thu, Jun 10, 2021 at 2:00 PM Nikhil Mohite <
>> nikhil(dot)mohite(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Dave,
>>>
>>> On Thu, Jun 10, 2021 at 5:22 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>
>>>> Hi
>>>>
>>>> On Thu, Jun 10, 2021 at 11:08 AM Nikhil Mohite <
>>>> nikhil(dot)mohite(at)enterprisedb(dot)com> wrote:
>>>>
>>>>> Hi Dave/ Team,
>>>>>
>>>>> We are facing an issue with winpty.dll on Windows server 2016 and
>>>>> Windows 7(these are platforms on which we have tested).
>>>>> Files required for winpty are present in the site-packages but still,
>>>>> it is unable to load the winpty.dll file on these specific platforms. We
>>>>> have tested it on Windows 10 pro and Windows server 2019 and it is working
>>>>> fine. (Also tried building the local pywinpty but unable to build it.) ref
>>>>> link for winPty
>>>>> https://github.com/rprichard/winpty#:~:text=winpty%20is%20a%20Windows%20software,in%20a%20Cygwin%2FMSYS%20pty.
>>>>>
>>>>
>>>> Does winpty.exe run, if executed from the command line instead of
>>>> pgAdmin? If not, does dependency walker show any missing libraries that are
>>>> required?
>>>>
>>> Not found the winpty.exe in site packages, but as per the winpty
>>> documents "winpty-agent.exe" will start the process with a new, hidden
>>> console window. It is not showing any error while installing the package. I
>>> tried to use it outside the pgAdmin but still facing the same error
>>> "Exception in import winpty DLL load failed while importing winpty: The
>>> specified procedure could not be found."(created a separate python
>>> environment for this)
>>>
>>> 1. if try to run winpty-agent.exe from the command line, it shows the
>>> user entered inputs on the same terminal, not showing any errors.
>>> [image: image.png]
>>>
>>> 2. I found winpty.exe in the other installed app (Git command line:
>>> C:\Program Files (x86)\Git\usr\bin) if try to run it through the command
>>> line not getting any error, it is showing user entered inputs on the same
>>> terminal. (It is not related to pgAdmin but just tried to check winpty.exe
>>> throwing any error or not)
>>> [image: image.png]
>>>
>>
>> Hmmm, that sounds oddly similar to an issue I had with Kerberos on
>> Windows when I was mucking around with that. I can't remember the exact
>> details, but as a test, does the problem go away if winpty.dll is copied
>> into the Windows system32 directory?
>>
> Tried it but still facing the same issue.
>

OK, I had a fiddle around, and see the same thing. I can't do a simple
import of winpty. On 2019 it's fine. I tried various tricks that I've used
in the past to make sure libraries are found etc.

Could this be because it's trying to use the conpty interface in Windows 10
and above?

If we can't get this to work tomorrow, I think it's fine to simply hide the
menu options and button if the server detects it's running on an
unsupported version of Windows. Of course, this should be documented. We
also need to make sure we know *exactly* what versions it does and does not
work on.

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

EDB: https://www.enterprisedb.com

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Nikhil Mohite 2021-06-10 15:46:02 Re: [pgAdmin][RM-2341]: Add menu option for starting PSQL
Previous Message Nikhil Mohite 2021-06-10 13:22:43 Re: [pgAdmin][RM-2341]: Add menu option for starting PSQL