git: 8d831eb43aba - main - databases/postgresql??-server: rc-script should require DAEMON

[Mateusz Piotrowski]

On 01/06/2021 15:59, Palle Girgensohn wrote:
> The branch main has been updated by girgen:
>
> URL: https://cgit.FreeBSD.org/ports/commit/?id=8d831eb43aba59013b948830cbd4b22b45deb223
>
> commit 8d831eb43aba59013b948830cbd4b22b45deb223
> Author:     Palle Girgensohn <girgen at FreeBSD.org>
> AuthorDate: 2021-06-01 13:59:02 +0000
> Commit:     Palle Girgensohn <girgen at FreeBSD.org>
> CommitDate: 2021-06-01 13:59:35 +0000
>
>      databases/postgresql??-server: rc-script should require DAEMON
>      
>      Changeset ab83f2b4bb78 changed the startup order for Postgresql. The cleartmp
>      rc.d now comes after the Postgresql startup.  Unfortunately, Postgresql likes
>      to create a socket in /tmp/.s.PGSQL.5432. After cleartmp does its work, that
>      socket disappears from the filesystem.
>      
>      Submitted by:   Jeroen Pulles
>      PR:             256335
> ---
[snip]
>   
> diff --git a/databases/postgresql10-server/files/postgresql.in b/databases/postgresql10-server/files/postgresql.in
> index 88ace679b95c..50725fd59103 100644
> --- a/databases/postgresql10-server/files/postgresql.in
> +++ b/databases/postgresql10-server/files/postgresql.in
> @@ -1,7 +1,7 @@
>   #!/bin/sh
>   
>   # PROVIDE: postgresql
> -# REQUIRE: SERVERS
> +# REQUIRE: DEAMON

Shouldn't it also explicitly say REQUIRE: cleartmp?