From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Muhammad Aqeel <muhammad(dot)aqeel(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: PIP Package Building for pgAdmin4 |
Date: | 2016-03-09 17:25:04 |
Message-ID: | CA+OCxoyqtQW_gG_VMoB0AU-ksqK7b-PQAp0cahP5v2-+X6NN6Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi
On Wed, Mar 9, 2016 at 2:24 PM, Muhammad Aqeel
<muhammad(dot)aqeel(at)enterprisedb(dot)com> wrote:
> Hi,
>
> PFA the patch to build PIP package for pgAdmin4 web component. It builds PIP
> wheel file based on Python version in PATH.
>
> This pip package build setup is supported for python version 2.6 and above.
> The pip package can easily be built with a make statement.
>
> Makefile has following targets.
> - install-required: To install packages required to build PIP package.
> - all: To build PIP package
> - install-pip: To install built PIP package
> - clean: To clean the sources.
Let's try to keep the Makefile target names specific to what we're
building, e.g.
install-pip-requirements: To install packages required to build PIP
package (not sure that's the best name though)
install-pip: To install built PIP package
pip: To build the PIP package
Eventually the "all" target will likely build the runtime and maybe
other packages as well, so for now, it should probably look something
like:
all: install-pip-requirements pip
Similarly, "clean" will eventually do other things too, so we can
create "clean-pip", which can be called with other further clean
targets by "clean".
> setup_pgadmin4.py file contains all the Meta information needed to build the
> PIP package.
I think that should be moved to pkg/pip, where it can later live
alongside directories like win32, osx and rpm etc. Would setup_pip.py
be a better name as well?
> The version for the pip package is picked up from web/config.py file.
Urgh - don't try to parse that file from a Python script. It's a
Python script itself, so you can just import it - e.g.
import config
version = config.APP_VERSION
Thanks!
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2016-03-09 17:34:38 | Re: [pgAdmin4] [Patch]: Grant Wizard |
Previous Message | Dave Page | 2016-03-09 17:12:28 | Re: PATCH: Updated ACL parsing function. [pgAdmin4] |