Re: PIP Package Building for pgAdmin4 - Mailing list pgadmin-hackers

From Muhammad Aqeel
Subject Re: PIP Package Building for pgAdmin4
Date
Msg-id CAJ4pYuxdjrtexxSbz=2cdw8saO04YKcXmXfeoEQZU0-Jw3diWQ@mail.gmail.com
Whole thread Raw
In response to Re: PIP Package Building for pgAdmin4  (Dave Page <dpage@pgadmin.org>)
Responses Re: PIP Package Building for pgAdmin4
List pgadmin-hackers
PFA the updated patch.

On Wed, Mar 9, 2016 at 10:25 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Wed, Mar 9, 2016 at 2:24 PM, Muhammad Aqeel
<muhammad.aqeel@enterprisedb.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

Done. 

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

Done. 

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".

Done.
 

> 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?

Done.
 

> 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

Done.
 

Thanks!

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

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

Attachment

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Fix a number of minor issues with catalog objects:
Next
From: Dave Page
Date:
Subject: Re: [pgAdmin4][Patch]: Foreign Data Wrapper