All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
blob 573eb413b040a309cf19055b086f2d3cdca25739 2186 bytes (raw)
name: meta/recipes-connectivity/ppp/ppp/0001-Revert-lock-path-to-var-lock-435.patch 	 # note: path name is non-authoritative(*)

 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
 
From 99cbf5e269994482edaf64624be8b1c806f9587c Mon Sep 17 00:00:00 2001
From: Dominique Martinet <asmadeus@codewreck.org>
Date: Tue, 10 Oct 2023 10:05:50 +0900
Subject: [PATCH] Revert lock path to /var/lock (#435)

lock dir changed on linux from /var/lock to /run/pppd/lock with
pppd-2.5.0, which makes pppd fail to start if the distribution does not
pre-create the directory.

This reverts it back to /var/lock.

The paths for other OS should be identical as LOCALSTATEDIR should be
/var, but also revert them back as well just in case.
Since the variable is no longer used remove it from makefiles.

Fixes: 66a8c74c3f73 ("Let ./configure control the paths for pppd")
Fixes: #419

Signed-off-by: Dominique Martinet <dominique.martinet@atmark-techno.com>
Co-authored-by: Dominique Martinet <dominique.martinet@atmark-techno.com>

Upstream-Status: Backport [https://github.com/ppp-project/ppp/commit/99cbf5e269994482edaf64624be8b1c806f9587c]
---
 pppd/Makefile.am | 2 +-
 pppd/pathnames.h | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/pppd/Makefile.am b/pppd/Makefile.am
index e5bedf2..7cb3005 100644
--- a/pppd/Makefile.am
+++ b/pppd/Makefile.am
@@ -83,7 +83,7 @@ pppd_SOURCES = \
     upap.c \
     utils.c
 
-pppd_CPPFLAGS = -DSYSCONFDIR=\"${sysconfdir}\" -DLOCALSTATEDIR=\"${localstatedir}\" -DPPPD_RUNTIME_DIR='"@PPPD_RUNTIME_DIR@"' -DPPPD_LOGFILE_DIR='"@PPPD_LOGFILE_DIR@"'
+pppd_CPPFLAGS = -DSYSCONFDIR=\"${sysconfdir}\" -DPPPD_RUNTIME_DIR='"@PPPD_RUNTIME_DIR@"' -DPPPD_LOGFILE_DIR='"@PPPD_LOGFILE_DIR@"'
 pppd_LDFLAGS =
 pppd_LIBS =
 
diff --git a/pppd/pathnames.h b/pppd/pathnames.h
index de2fb68..12609a9 100644
--- a/pppd/pathnames.h
+++ b/pppd/pathnames.h
@@ -120,12 +120,12 @@
 #define PPP_PATH_PPPDB          PPP_PATH_VARRUN  "/pppd2.tdb"
 
 #ifdef __linux__
-#define PPP_PATH_LOCKDIR        PPP_PATH_VARRUN  "/lock"
+#define PPP_PATH_LOCKDIR        "/var/lock"
 #else
 #ifdef SVR4
-#define PPP_PATH_LOCKDIR        LOCALSTATEDIR "/spool/locks"
+#define PPP_PATH_LOCKDIR        "/var/spool/locks"
 #else
-#define PPP_PATH_LOCKDIR        LOCALSTATEDIR "/spool/lock"
+#define PPP_PATH_LOCKDIR        "/var/spool/lock"
 #endif
 #endif
 
-- 
2.43.0


debug log:

solving 573eb413b0 ...
found 573eb413b0 in https://yhbt.net/lore/all/20250213114203.2056946-1-johannes.schneider@leica-geosystems.com/ ||
	https://yhbt.net/lore/all/20250214075348.3834651-1-johannes.schneider@leica-geosystems.com/ ||
	https://yhbt.net/lore/all/183d2ff42636afeb4fb2394dfa17ab9c2c0620ce.1739913090.git.steve@sakoman.com/

applying [1/1] https://yhbt.net/lore/all/20250213114203.2056946-1-johannes.schneider@leica-geosystems.com/
diff --git a/meta/recipes-connectivity/ppp/ppp/0001-Revert-lock-path-to-var-lock-435.patch b/meta/recipes-connectivity/ppp/ppp/0001-Revert-lock-path-to-var-lock-435.patch
new file mode 100644
index 0000000000..573eb413b0

1:41: trailing whitespace.
 
1:46: trailing whitespace.
 
1:53: trailing whitespace.
 
1:66: trailing whitespace.
 
1:67: trailing whitespace.
-- 
Checking patch meta/recipes-connectivity/ppp/ppp/0001-Revert-lock-path-to-var-lock-435.patch...
Applied patch meta/recipes-connectivity/ppp/ppp/0001-Revert-lock-path-to-var-lock-435.patch cleanly.
warning: squelched 1 whitespace error
warning: 6 lines add whitespace errors.

skipping https://yhbt.net/lore/all/20250214075348.3834651-1-johannes.schneider@leica-geosystems.com/ for 573eb413b0
skipping https://yhbt.net/lore/all/183d2ff42636afeb4fb2394dfa17ab9c2c0620ce.1739913090.git.steve@sakoman.com/ for 573eb413b0
index at:
100644 573eb413b040a309cf19055b086f2d3cdca25739	meta/recipes-connectivity/ppp/ppp/0001-Revert-lock-path-to-var-lock-435.patch

(*) Git path names are given by the tree(s) the blob belongs to.
    Blobs themselves have no identifier aside from the hash of its contents.^

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.