Re: Cleanup runtime qmake code - Mailing list pgadmin-hackers

From Dave Page
Subject Re: Cleanup runtime qmake code
Date
Msg-id CA+OCxozb25DpbCj4PLJnhEBQyhYhUV4ixwLANkEoduyCoY5Stw@mail.gmail.com
Whole thread Raw
In response to Re: Cleanup runtime qmake code  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
List pgadmin-hackers
Hi

On Mon, May 4, 2020 at 10:03 AM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Dave

Patch looks good to me except one small change in the README file that is PGADMIN_PYTHON_PATH is mentioned but it should be PGADMIN_PYTHON_DIR

I kept mixing those up :-(

I've committed the patch, and made a couple of changes to Jenkins jobs where I thought necessary. Hopefully nothing breaks too much!

Thanks!
 

On Fri, May 1, 2020 at 8:29 PM Dave Page <dpage@pgadmin.org> wrote:
The attached patch tidies up the qmake code for configuring the runtime build so that it's consistent on all platforms, and simply requires that the PGADMIN_PYTHON_DIR environment variable points to the Python installation to be used.

Please review, but don't commit, as this change will probably break some of the Jenkins jobs. I'll commit and fix them when the patch has passed review.

(Sidenote: this is part of a larger tidy up/standardisation of the build scripts).

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

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


--
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246


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

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

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: Re: [pgAdmin 4 - Housekeeping #5255] Implement Selenium Grid usingmulti-threading & solenoid using current test framework
Next
From: Dave Page
Date:
Subject: Re: Patch to remove Python 2.7 support in the macOS Appbundle