Message-ID: <1476102053.2867.39.camel@gunduz.org>
Subject: Re: Running pgadmin4 as desktop application
From: Devrim =?ISO-8859-1?Q?G=FCnd=FCz?= <devrim@gunduz.org>
To: Dave Page <dpage@pgadmin.org>
Cc: pgadmin-hackers <pgadmin-hackers@postgresql.org>
Date: Mon, 10 Oct 2016 15:20:53 +0300
In-Reply-To: 
 <CA+OCxoyr1K1HKHgddVf-heHyDpbzfk_yWQaUki72sATqsFBhNA@mail.gmail.com>
References: <1475749506.3515.12.camel@gunduz.org>
	 <CA+OCxoyr1K1HKHgddVf-heHyDpbzfk_yWQaUki72sATqsFBhNA@mail.gmail.com>
Content-Type: multipart/signed; micalg="pgp-sha256";
	protocol="application/pgp-signature"; boundary="=-Z1ncw4JtbJggG0zE9n44"
Mime-Version: 1.0
Precedence: bulk
Sender: pgadmin-hackers-owner@postgresql.org


--=-Z1ncw4JtbJggG0zE9n44
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable


Hi Dave,

On Thu, 2016-10-06 at 13:42 +0100, Dave Page wrote:
>=20
> The global config file that Qt will read by default seems to be
> "/etc/pgAdmin Development Team/pgAdmin 4.conf".
>=20
> I may change that for 1.1 to use a less obnoxious path. The contents
> of it seem to need to be:
>=20
> [General]
> ApplicationPath=3D../../../../usr/lib/python2.7/site-packages/pgadmin4-we=
b/
> PythonPath=3D"/usr/lib64/python2.7/site-packages/;/usr/lib/python2.7/site=
-
> packages/"
>=20
> The ApplicationPath is the directory containing pgAdmin4.py. Due to a
> bugette in the way the runtime works, it needs to be relative to the
> location of the runtime, hence all the back-tracking. The PythonPath
> needs to include both the 32 and 64 bit directories, as it seems that
> some of the Python modules end up in each.
>=20
> That was enough to get it to start for me.

Ok, this worked on CentOS 7 box, but not on Fedora 24, which has QT5 and Py=
thon
3. Can you help me to debug this? The content of the file in Fedora 24 is:

$ cat /etc/pgAdmin\ Development\ Team/pgAdmin\ 4.conf=C2=A0
[General]
ApplicationPath=3D../../../..//usr/lib/python3.5/site-packages/pgadmin4-web=
/
PythonPath=3D"/usr/lib64/python3.5/site-packages; /usr/lib/python3.5/site-p=
ackages"


I'm still getting segfault, and:

An error occurred initialising the application server:

Failed to locate pgAdmin4.py, terminating server thread.


> Then, we run into the tricky issues, where you may have to deal with
> conflicts between web and desktop mode. In our other installers, we
> ship a blank config_local.py file, with a simple comment at the top
> telling the user they can customise any settings in that file. For web
> users, we don't want that, as it's used as a security mechanism
> (startup will fail if it's not present, and tell the user they must
> set some security-critical options). Those options don't matter in
> desktop mode, hence why we pre-write the file.
>=20
> Perhaps the way around that is to have the shortcut on the menu call a
> script that can create config_local.py if needed? Though, I see you
> already have config_distro.py forcing the user into desktop mode,
> which is a potential conflict.

AFAICS, creating the file above helped to solve this issue, at least I can =
say
that both modes work on CentOS 7.

> The other issue I see is with the docs:
>=20
> 1) The path in config_distro.py is
> /usr/share/doc/pgadmin4-docs/en_US/html/. That directory doesn't exist
> - if the docs are compiled in-situ, they'll end up in
> /usr/share/doc/pgadmin4-docs/en_US/html/_build/html
>=20
> 2) The docs aren't built, so there is no _build/html directory.

Actuallly they are installed via -docs subpackage, but I forgot to add this=
 as
a dependency.

> 3) The docs are integrated into the application so really should be a
> dependency of the pgadmin4 RPM.

Ok, fixed. Pushed to 9.6 testing repo:

yum =C2=A0--enablerepo pgdg96-updates-testing install pgadmin4 pgadmin4-web

(Packages will appear in next sync ;) )

Can you please test again?

Cheers,

--=20
Devrim G=C3=9CND=C3=9CZ
EnterpriseDB: http://www.enterprisedb.com
PostgreSQL Dan=C4=B1=C5=9Fman=C4=B1/Consultant, Red Hat Certified Engineer
Twitter: @DevrimGunduz , @DevrimGunduzTR

--=-Z1ncw4JtbJggG0zE9n44
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: This is a digitally signed message part
Content-Transfer-Encoding: 7bit

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQIcBAABCAAGBQJX+4elAAoJENjDPf6Sz261NicQAKk7SW+x0CbQ6PD9npen9iA2
JPJ48Q+PMv3S8/806eAtv0MMKmj5JSiC/70058+GC08mU0YwJFB/qe11D3gU6XoO
nu8ZcpT2bovLYM9282vZsPw12h1ALp8SSMTeXxgC1S4aJvKH5pVzLSm+GF05KWeY
CKnXLAhvGYh1pQVNCKNYNdsp6p+PYMZn7It69BtsX8dEe2BWSCte3RbDmFQPYg2o
Ld5qo+lokxtJpRasnLocHgeknYY1BsEXCRt+qOLoue0GM4MgYVXxL/paOMSyFU68
fVYbLycirroFgx0jmrDwqFKr0Pj0qr8fnx5uFj1GLOKLwVqRm6ujxZgp+Ttvp4jn
sJR9Ja15Zj2mtUzpsFwrqYEx2Urj63XeW+RhBILMR7oGkewICpsUj/U2I4yP8mVV
g3Wa9D71pEEKo30YoD/hvEdT2xrP8w2T6VRx31FrMy7MgbstEBXFzo7TgY7FRhFu
PDahIn7J26m68UN/Dvc2pWAlti1l/sRE2qxvwbZuBW8m+neEDnwWmHDji7p3h8BO
4Nv7Kf4YD4/pFAHQ52Bkx7POxlgcPa9OJGrHSHS8iKYpFnG7OmWJy/NfQFLYjBK8
aaONOE2uQrbrLtMyHosJ3WeHQmXaObaekA4XEpYVFv8Ok3W9ivK0CN2qTd8bO1oB
qKJEv2aR1eZ0HyzIScy5
=vMmG
-----END PGP SIGNATURE-----

--=-Z1ncw4JtbJggG0zE9n44--