On Mon, May 8, 2017 at 11:57 AM, Devrim Gündüz <devrim@gunduz.org> wrote:
Hi Josh,
Sorry for the late response, I have 10K unread emails as of now: On Tue, 2017-04-25 at 08:24 -0700, Josh Berkus wrote:
> I know how to fix my personal install (and container image). But I want > to fix the RPM package so that it's fixed for most users. Right now, > install from RPM is kind of broken. > > Here's the changes that need to happen on the RPM: > > 1. Remove the QT dependencies for pgadmin4-v1-web
Why? I thought we need them to run pgadmin4?
Only in desktop mode. For web-only use, no Qt should be needed.
> 2. Have the RPM create /usr/share/httpd/.pgadmin, owned by apache user. > (is this the best location? If not, what is?)
Good question. I think that location is good.
> 3. Fix the systemd unit file so that the pgadmin4 unit is owned by apache.
Why? I did not read all the emails in this thread (yet), but all of the unit files are owned by root, and I see no exception on my system. > 4. (optional) set up files according to FDL standards, putting configs > for pgadmin4 in /etc/pgadmin and the db in /var/run/pgadmin (or similar).
Red Hat does not care about that that much, does it? Especially for /etc part.
Debian does though. That's been on my TODO list for a while. It's pretty easy to do though - we can have a config option in config.py to specify the location of config_local.py, defaulting to the current location. Packagers could then override that in config_distro.py.