Gentoo Archives: gentoo-dev

From: Michael Orlitzky <mjo@g.o>
To: gentoo-dev@l.g.o
Subject: Re: [gentoo-dev] [PATCH] eclass/postgres.eclass: migrate to GLEP 81
Date: Thu, 22 Jul 2021 22:35:47
Message-Id: 1e1d24487f4adbc0c2e0bbe982567d3bad6796cc.camel@gentoo.org
In Reply to: [gentoo-dev] [PATCH] eclass/postgres.eclass: migrate to GLEP 81 by Conrad Kostecki
1 On Fri, 2021-07-23 at 00:20 +0200, Conrad Kostecki wrote:
2 > This update drops the function 'postgres_new_user', which was used to
3 > create the 'postgres' user and group.
4 >
5 > ...
6 >
7 > Since many other packages depend on the 'postgres' and 'postgres-multi'
8 > eclass, adding the core acct-*/postgres packages here to [R]DEPEND.
9
10 Not all packages using the eclass necessarily need the "postgres"
11 user/group to build/run. You should only add (R)DEPEND in the packages
12 that actually call postgres_new_user, I think?
13
14
15 >
16 > Before merging this eclass patch, acct-* packages will be added to
17 > the tree.
18
19 Before doing this, please double-check that SHELL=/bin/bash and
20 HOME=/var/lib/postgres are needed. It's likely that the default HOME
21 can be used, and that dev-db/postgresql (which actually *uses*
22 /var/lib/postgres) should control its permissions.
23
24 You should also get an ACK from titanofold, since he's been maintaining
25 postgresql essentially by himself forever (and may have some insight
26 into the SHELL/HOME items).