[PATCH] doc: add comment about ofl for Pam mount.

  • Done
  • quality assurance status badge
Details
2 participants
  • Ludovic Courtès
  • Michael Rohleder
Owner
unassigned
Submitted by
Michael Rohleder
Severity
normal
M
M
Michael Rohleder wrote on 3 May 2020 14:42
(address . guix-patches@gnu.org)
87ftch42bx.fsf@rohleder.de
Pam mount uses `ofl` from hxtools [1] to kill processes that have open
files on the filesystem mounted.
As we dont have hxtools (and I think we shouldnt), one has to configure
pam mount to use something else, eg `fuser`. This is an attempt to
document that somehow.
From 7a3cd9cd00cbd95e599a8d79400591e2fc589f33 Mon Sep 17 00:00:00 2001
From: Michael Rohleder <mike@rohleder.de>
Date: Sun, 3 May 2020 14:17:37 +0200
Subject: [PATCH] doc: add comment about ofl for Pam mount.

* doc/guix.texi: (Pam Mount) add comment about ofl.
---
doc/guix.texi | 2 ++
1 file changed, 2 insertions(+)

Toggle diff (15 lines)
diff --git a/doc/guix.texi b/doc/guix.texi
index d5d8662937..aa182df183 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -25441,6 +25441,8 @@ the partition where he stores his data:
(hup "0")
(term "no")
(kill "no")))
+ ;; if anything above is "true":
+ ;; (ofl "/run/current-system/profile/bin/fuser -km %(MNTPT)")
(mkmountpoint (@@ (enable "1")
(remove "true")))))
--
2.26.2
Footnotes:

--
"These download files are in Microsoft Word 6.0 format. After unzipping,
these files can be viewed in any text editor, including all versions of
Microsoft Word, WordPad, and Microsoft Word Viewer." [Microsoft website]
-----BEGIN PGP SIGNATURE-----

iQEzBAEBCAAdFiEEdV4t5dDVhcUueCgwfHr/vv7yyyUFAl6uvDIACgkQfHr/vv7y
yyWTuggAlu3Bh4/ELKphaUnHm9y/FLRgMKWWBjHVOAGwUEMSAbgK0W62yg/59FXV
jCyGw6jvdgN+eGn2lTr+wSYe+BfGCTF4a2rMXZohe7rLCdHlQG6obgtQ8twvu4m2
DOGNOdTLp/BlWFgGRvDgOa0U4z9q0QZHFVFRFYuklqwtYCAb+j6gzpsWrpvvqsQz
SuyzKV09huOqVGcV4hhvoFTr67hv/xzj2MzaulNQVeIcEF4rcLjsYrDAqo7ZVOXk
RBB08IMyAY3nPbabI0JNOXKKwyqToDFCeezZjlJH0+D1lmZiRlYy4hIlW4Cd/fRj
4cCLUHPs0GL9fJYRguMR27itEY76Rw==
=zD3D
-----END PGP SIGNATURE-----

L
L
Ludovic Courtès wrote on 2 Sep 2020 16:49
(name . Michael Rohleder)(address . mike@rohleder.de)(address . 41048@debbugs.gnu.org)
87lfhs5j3x.fsf@gnu.org
Hi,

This patch fell through the cracks…

Michael Rohleder <mike@rohleder.de> skribis:

Toggle quote (27 lines)
> Pam mount uses `ofl` from hxtools [1] to kill processes that have open
> files on the filesystem mounted.
> As we dont have hxtools (and I think we shouldnt), one has to configure
> pam mount to use something else, eg `fuser`. This is an attempt to
> document that somehow.
>
> From 7a3cd9cd00cbd95e599a8d79400591e2fc589f33 Mon Sep 17 00:00:00 2001
> From: Michael Rohleder <mike@rohleder.de>
> Date: Sun, 3 May 2020 14:17:37 +0200
> Subject: [PATCH] doc: add comment about ofl for Pam mount.
>
> * doc/guix.texi: (Pam Mount) add comment about ofl.
> ---
> doc/guix.texi | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/doc/guix.texi b/doc/guix.texi
> index d5d8662937..aa182df183 100644
> --- a/doc/guix.texi
> +++ b/doc/guix.texi
> @@ -25441,6 +25441,8 @@ the partition where he stores his data:
> (hup "0")
> (term "no")
> (kill "no")))
> + ;; if anything above is "true":
> + ;; (ofl "/run/current-system/profile/bin/fuser -km %(MNTPT)")

It’s not clear what these two lines mean. The idea is to tell users to
configure PAM Mount to use ‘fuser’ as a replacement for ‘ofl’, right? I
think that’d deserve one or two sentences of explanation. WDYT?

Thanks,
Ludo’.
M
M
Michael Rohleder wrote on 2 Sep 2020 17:25
(name . Ludovic Courtès)(address . ludo@gnu.org)(address . 41048@debbugs.gnu.org)
874kogciao.fsf@rohleder.de
Hi Ludo!

Ludovic Courtès <ludo@gnu.org> writes:
Toggle quote (7 lines)
>> + ;; if anything above is "true":
>> + ;; (ofl "/run/current-system/profile/bin/fuser -km %(MNTPT)")
>
> It’s not clear what these two lines mean. The idea is to tell users to
> configure PAM Mount to use ‘fuser’ as a replacement for ‘ofl’, right? I
> think that’d deserve one or two sentences of explanation. WDYT?

Yes, that is/was the idea, because killing processes would not work
otherwise.
Now that we have autofs, I don't think it's relevant any more, so we can
close it?

--
No one gets sick on Wednesdays.
-----BEGIN PGP SIGNATURE-----

iQFFBAEBCAAvFiEEdV4t5dDVhcUueCgwfHr/vv7yyyUFAl9PuV8RHG1pa2VAcm9o
bGVkZXIuZGUACgkQfHr/vv7yyyW5dwf/a08C/6L4sXbMzprA9kHqwbcHasa8RnMP
Hd7O3wk3vc/Hr9ghJ/sgLLZHN+jGDOue8TMZJ2P80bVrqcTTYcnZEEhkaFSPPJV0
D2LgPhhAs1GKWLXH8h7f5fnWmzQ/u8mGB8LwdwGFqMPqfUbFtK8J2S8R5FPXkFkw
F3Jiuyddy7qJsg/wr/H34UBywdNzp+uGthqtaxZyGuoi+JaJfhx5rLJf4fJfQvei
r0ov5u/ecJRol9ORGq5qLUjqRv61UjeenBIAvhrIXI8XdBX/5EL7LNA9KByveRcz
/R0AscZoHAA+MVLHmno/PLDfuMGUTpVKFIHP+hQCYXIEDPsmOHxpPA==
=Gxew
-----END PGP SIGNATURE-----

L
L
Ludovic Courtès wrote on 2 Sep 2020 22:18
(name . Michael Rohleder)(address . mike@rohleder.de)(address . 41048-done@debbugs.gnu.org)
87y2ls2aql.fsf@gnu.org
Hi Michael,

Michael Rohleder <mike@rohleder.de> skribis:

Toggle quote (11 lines)
> Ludovic Courtès <ludo@gnu.org> writes:
>>> + ;; if anything above is "true":
>>> + ;; (ofl "/run/current-system/profile/bin/fuser -km %(MNTPT)")
>>
>> It’s not clear what these two lines mean. The idea is to tell users to
>> configure PAM Mount to use ‘fuser’ as a replacement for ‘ofl’, right? I
>> think that’d deserve one or two sentences of explanation. WDYT?
>
> Yes, that is/was the idea, because killing processes would not work
> otherwise.

OK.

Toggle quote (3 lines)
> Now that we have autofs, I don't think it's relevant any more, so we can
> close it?

Fine with me, done!

Thanks,
Ludo’.
Closed
?