From: | Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Shaheed Haque <shaheedhaque(at)gmail(dot)com> |
Subject: | Re: Making Kerberos optional in the Python wheel |
Date: | 2021-03-08 11:20:49 |
Message-ID: | CAFOhELdD6R4-iAux02qjwAK3Ox4ovGbkPy-kVeoj71F3bN47Rw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Dave,
The patch looks good to me.
Thanks,
Khushboo
On Mon, Mar 8, 2021 at 4:42 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
> Has anyone been able to review this?
>
> On Thu, Mar 4, 2021 at 10:02 AM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> There have been a couple of complaints that the latest Python wheel
>> distribution doesn't install cleanly. This happens when there is no
>> pre-built gssapi wheel on PyPi that matches the users combination of Python
>> version and platform, *and* the MIT Kerberos development headers etc. are
>> not present on the system, so the source wheel cannot be compiled.
>>
>> This seems like it's a bit onerous on users, especially if they're on
>> Windows where they'll also need a suitable compiler to be installed. The
>> attached patch aims to address that by making the Kerberos support optional
>> (thankfully, Khushboo made the code handle lack of gssapi libraries).
>>
>> To install without gssapi, users would simply do:
>>
>> pip install pgadmin4
>>
>> or
>>
>> pip install pip install /path/to/pgadmin4-5.0-py3-none-any.whl
>>
>> To install with gssapi:
>>
>> pip install pgadmin4['kerberos']
>>
>> or
>>
>> pip install pip install /path/to/pgadmin4-5.0-py3-none-any.whl['kerberos']
>>
>> The patch also cleans up some old cruft that was required for now
>> unsupported Python versions.
>>
>> Thoughts?
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EDB: http://www.enterprisedb.com
>>
>>
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EDB: http://www.enterprisedb.com
>
>
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2021-03-08 11:33:09 | pgAdmin 4 commit: Make 'kerberos' an optional feature in the Python whe |
Previous Message | Dave Page | 2021-03-08 11:19:42 | pgAdmin 4 commit: Ensure our venv uses the correct python interpreter, |