diff mbox series

[rpcbind,1/1] Move rpcbind.lock to /run

Message ID 20240830173920.88877-1-pvorel@suse.cz (mailing list archive)
State New
Headers show
Series [rpcbind,1/1] Move rpcbind.lock to /run | expand

Commit Message

Petr Vorel Aug. 30, 2024, 5:39 p.m. UTC
From: Thomas Blume <thomas.blume@suse.com>

Most of the distros have /var/run as symlink to /run.

Because /var may be a separate partition, and could even be mounted via
NFS, having to look directly to /run help to avoid issues rpcbind
startup early in boot when /var might not be available.

Reviewed-by: Petr Vorel <pvorel@suse.cz>
Signed-off-by: Thomas Blume <thomas.blume@suse.com>
Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
NOTE: I chose opensuse patch for the simplicity, instead of Debian
patch, which unsets _PATH_RPCBINDSOCK (libtirpc).

I'll send a separate patch for libtirpc.

Kind regards,
Petr

 src/rpcbind.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Steve Dickson Aug. 31, 2024, 5:55 p.m. UTC | #1
On 8/30/24 1:39 PM, Petr Vorel wrote:
> From: Thomas Blume <thomas.blume@suse.com>
> 
> Most of the distros have /var/run as symlink to /run.
> 
> Because /var may be a separate partition, and could even be mounted via
> NFS, having to look directly to /run help to avoid issues rpcbind
> startup early in boot when /var might not be available.
> 
> Reviewed-by: Petr Vorel <pvorel@suse.cz>
> Signed-off-by: Thomas Blume <thomas.blume@suse.com>
> Signed-off-by: Petr Vorel <pvorel@suse.cz>
Committed... (tag: rpcbind-1_2_8-rc1)

steved.
> ---
> NOTE: I chose opensuse patch for the simplicity, instead of Debian
> patch, which unsets _PATH_RPCBINDSOCK (libtirpc).
> 
> I'll send a separate patch for libtirpc.
> 
> Kind regards,
> Petr
> 
>   src/rpcbind.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/rpcbind.c b/src/rpcbind.c
> index ecebe97..9887b82 100644
> --- a/src/rpcbind.c
> +++ b/src/rpcbind.c
> @@ -105,7 +105,7 @@ char *nss_modules = "files";
>   /* who to suid to if -s is given */
>   #define RUN_AS  "daemon"
>   
> -#define RPCBINDDLOCK "/var/run/rpcbind.lock"
> +#define RPCBINDDLOCK "/run/rpcbind.lock"
>   
>   int runasdaemon = 0;
>   int insecure = 0;
diff mbox series

Patch

diff --git a/src/rpcbind.c b/src/rpcbind.c
index ecebe97..9887b82 100644
--- a/src/rpcbind.c
+++ b/src/rpcbind.c
@@ -105,7 +105,7 @@  char *nss_modules = "files";
 /* who to suid to if -s is given */
 #define RUN_AS  "daemon"
 
-#define RPCBINDDLOCK "/var/run/rpcbind.lock"
+#define RPCBINDDLOCK "/run/rpcbind.lock"
 
 int runasdaemon = 0;
 int insecure = 0;