Re: systemd service files vs. postgresql9x-setup

From: Strahinja Kustudić <strahinjak(at)nordeus(dot)com>
To: Bernd Helmle <mailings(at)oopsware(dot)de>
Cc: Devrim GÜNDÜZ <devrim(at)gunduz(dot)org>, Jeff Frost <jeff(at)pgexperts(dot)com>, pgsql-pkg-yum <pgsql-pkg-yum(at)postgresql(dot)org>
Subject: Re: systemd service files vs. postgresql9x-setup
Date: 2016-05-18 15:06:30
Message-ID: CADKbJJWtk_wh-SF5rrsLxNUvgmd2FnzShQGvj7-dwTa5VRGqRw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-pkg-yum

I propose the same patch with a modification to work with spaces in the
path, since systemctl prints spaces as '\x20', so we just needed one more
sed to replace that with a space.

Would also like to see this fixed in the next minor release, since it makes
it really hard to change PGDATA the correct way, which is by droping a
.conf file into /etc/systemd/system/postgresql-9.x.service.d/ directory.

Also would be really cool if the postgres rpm created the
/etc/systemd/system/postgresql-9.x.service.d/ directory, but it's not that
important as the setup scripts working correctly.

Strahinja Kustudić | Lead System Engineer | Nordeus

On Fri, Mar 11, 2016 at 1:01 PM, Bernd Helmle <mailings(at)oopsware(dot)de> wrote:

>
>
> --On 18. September 2015 16:48:09 +0300 Devrim GÜNDÜZ <devrim(at)gunduz(dot)org>
> wrote:
>
> > First of all: Thanks for the patch!
> >
> > The next set of new minor releases are due in a few weeks. I think we
> > can apply this patch then.
> >
> > (We need a issue tracker!)
>
> Hmm, i've totally forgotten that. I see that it's not committed yet, so i
> just go forward and ask if there's something missing we need to address?
>
> --
> Thanks
>
> Bernd
>
>
> --
> Sent via pgsql-pkg-yum mailing list (pgsql-pkg-yum(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-pkg-yum
>

Attachment Content-Type Size
read-PGDATA-from-systemctl-and-space-in-path.patch text/x-patch 3.0 KB

In response to

Browse pgsql-pkg-yum by date

  From Date Subject
Next Message Kaname Furutani 2016-05-19 06:19:32 yum package for pg_repack 1.3.4
Previous Message Devrim Gündüz 2016-05-17 08:41:04 Re: Unsigned libevent RPM for 9.6 beta / F23