Re: Patch to remove Python 2.7 support in the macOS Appbundle

2020-05-04 Thread Dave Page
Thanks, applied. On Mon, May 4, 2020 at 10:59 AM Akshay Joshi wrote: > Hi Dave > > Patch looks good to me except the below line gets commented by mistake (if > i am not wrong) > > # ./complete-bundle.sh "${BUILDROOT}/${APP_BUNDLE_NAME}" || { echo > complete-bundle.sh failed; exit 1; } > > > On

Re: Patch to remove Python 2.7 support in the macOS Appbundle

2020-05-04 Thread Akshay Joshi
Hi Dave Patch looks good to me except the below line gets commented by mistake (if i am not wrong) # ./complete-bundle.sh "${BUILDROOT}/${APP_BUNDLE_NAME}" || { echo complete-bundle.sh failed; exit 1; } On Mon, May 4, 2020 at 3:02 PM Dave Page wrote: > Here's a rebased version. > > On Mon, Ma

Re: Patch to remove Python 2.7 support in the macOS Appbundle

2020-05-04 Thread Dave Page
Here's a rebased version. On Mon, May 4, 2020 at 7:33 AM Akshay Joshi wrote: > Hi Dave > > On Fri, May 1, 2020 at 5:21 PM Dave Page wrote: > >> Akshay, are you happy with this patch? >> > >The patch is not applied to the latest code. > >> >> On Mon, Apr 27, 2020 at 3:30 PM Dave Page wrote:

Re: Patch to remove Python 2.7 support in the macOS Appbundle

2020-05-03 Thread Akshay Joshi
Hi Dave On Fri, May 1, 2020 at 5:21 PM Dave Page wrote: > Akshay, are you happy with this patch? > The patch is not applied to the latest code. > > On Mon, Apr 27, 2020 at 3:30 PM Dave Page wrote: > >> >> >> On Mon, Apr 27, 2020 at 3:28 PM Khushboo Vashi < >> khushboo.va...@enterprisedb.co

Re: Patch to remove Python 2.7 support in the macOS Appbundle

2020-05-01 Thread Dave Page
Akshay, are you happy with this patch? On Mon, Apr 27, 2020 at 3:30 PM Dave Page wrote: > > > On Mon, Apr 27, 2020 at 3:28 PM Khushboo Vashi < > khushboo.va...@enterprisedb.com> wrote: > >> Hi, >> >> The code looks good to me. The package building was tested by Neel and we >> found that the venv

Re: Patch to remove Python 2.7 support in the macOS Appbundle

2020-04-27 Thread Dave Page
On Mon, Apr 27, 2020 at 3:28 PM Khushboo Vashi < khushboo.va...@enterprisedb.com> wrote: > Hi, > > The code looks good to me. The package building was tested by Neel and we > found that the venv was built with Python 3+ but the runtime was built > against Python 2.7. > As per Dave, Neel may have n

Re: Patch to remove Python 2.7 support in the macOS Appbundle

2020-04-27 Thread Khushboo Vashi
Hi, The code looks good to me. The package building was tested by Neel and we found that the venv was built with Python 3+ but the runtime was built against Python 2.7. As per Dave, Neel may have not used a venv of the correct version as it is working fine on Dav'e system. So, we can commit the pa

Re: Patch to remove Python 2.7 support in the macOS Appbundle

2020-04-27 Thread Khushboo Vashi
On Mon, Apr 27, 2020 at 3:28 PM Dave Page wrote: > > > On Mon, Apr 27, 2020 at 10:57 AM Khushboo Vashi < > khushboo.va...@enterprisedb.com> wrote: > >> >> >> On Mon, Apr 27, 2020 at 3:23 PM Dave Page wrote: >> >>> >>> >>> On Mon, Apr 27, 2020 at 10:52 AM Khushboo Vashi < >>> khushboo.va...@enter

Re: Patch to remove Python 2.7 support in the macOS Appbundle

2020-04-27 Thread Dave Page
On Mon, Apr 27, 2020 at 10:57 AM Khushboo Vashi < khushboo.va...@enterprisedb.com> wrote: > > > On Mon, Apr 27, 2020 at 3:23 PM Dave Page wrote: > >> >> >> On Mon, Apr 27, 2020 at 10:52 AM Khushboo Vashi < >> khushboo.va...@enterprisedb.com> wrote: >> >>> Hi Dave, >>> >>> As we support PostgreSQL

Re: Patch to remove Python 2.7 support in the macOS Appbundle

2020-04-27 Thread Khushboo Vashi
On Mon, Apr 27, 2020 at 3:23 PM Dave Page wrote: > > > On Mon, Apr 27, 2020 at 10:52 AM Khushboo Vashi < > khushboo.va...@enterprisedb.com> wrote: > >> Hi Dave, >> >> As we support PostgreSQL >=9.5, do we need to maintain SQL templates for >> older versions? >> > > Yes, for now. We're not activel

Re: Patch to remove Python 2.7 support in the macOS Appbundle

2020-04-27 Thread Dave Page
On Mon, Apr 27, 2020 at 10:52 AM Khushboo Vashi < khushboo.va...@enterprisedb.com> wrote: > Hi Dave, > > As we support PostgreSQL >=9.5, do we need to maintain SQL templates for > older versions? > Yes, for now. We're not actively removing support for old PostgreSQL versions, just not testing the

Re: Patch to remove Python 2.7 support in the macOS Appbundle

2020-04-27 Thread Khushboo Vashi
Hi Dave, As we support PostgreSQL >=9.5, do we need to maintain SQL templates for older versions? Thanks, Khushboo On Mon, Apr 27, 2020 at 3:16 PM Dave Page wrote: > The attached patch removes support for Python 2.7 in general from the > macOS appbundle build, thus also removing special handli

Patch to remove Python 2.7 support in the macOS Appbundle

2020-04-27 Thread Dave Page
The attached patch removes support for Python 2.7 in general from the macOS appbundle build, thus also removing special handling for use of the system Python installation. -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise