From: | Neel Patel <neel(dot)patel(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Raise an exception under Python < 3.4 |
Date: | 2020-04-29 09:30:20 |
Message-ID: | CACCA4P3nStvuuS2PYjRnsWHKVTq1aOc3vpw6LwRKcTDAi674uA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi,
On Wed, Apr 29, 2020 at 2:52 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
> Hi
>
> On Wed, Apr 29, 2020 at 10:13 AM Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
> wrote:
>
>> Hi Dave,
>>
>> Patch looks good to me except one condition as we do support from Python
>> 3.4 onwards. I have modified that condition and attached the updated patch.
>>
>
> What did you change exactly (curious about what I missed)?
>
Previously it was like,
*+if sys.version_info < (3, 3):*
*+ raise Exception('This application must be run under Python 3.4 or
later.')*
Modified as below.
*+if sys.version_info < (3, 4):*
*+ raise Exception('This application must be run under Python 3.4 or
later.')*
>
>>
>> I have also removed the other references of Python2. Attached is the
>> patch. Do review it and let me know for comments.
>>
>
> I think there's a lot more than that - try searching the entire tree for
> PY2 or sys.version. This was on my personal todo as a next step, but feel
> free to update your patch :-)
>
Sure.
>
> Thanks!
>
>
>>
>> Thanks,
>> Neel Patel
>>
>> On Mon, Apr 27, 2020 at 9:27 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>> The attached patch raises an exception if run under Python < 3.4. It
>>> also cleans up the README to remove references to Python 2 and removes
>>> Python 2-isms from the main config.
>>>
>>> --
>>> Dave Page
>>> Blog: http://pgsnake.blogspot.com
>>> Twitter: @pgsnake
>>>
>>> EnterpriseDB UK: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
From | Date | Subject | |
---|---|---|---|
Next Message | Yogesh Jain | 2020-04-29 11:56:03 | Re: [pgAdmin][RM4279] : Issue with File Browser Home button |
Previous Message | Dave Page | 2020-04-29 09:21:57 | Re: Raise an exception under Python < 3.4 |