[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

Debbugs page

Michael Rohleder wrote 5 years ago
(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-----

Ludovic Courtès wrote 5 years ago
(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’.
Michael Rohleder wrote 5 years ago
(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-----

Ludovic Courtès wrote 5 years ago
(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
?
Your comment

This issue is archived.

To comment on this conversation send an email to 41048@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 41048
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch
You may also tag this issue. See list of standard tags. For example, to set the confirmed and easy tags
mumi command -t +confirmed -t +easy
Or, remove the moreinfo tag and set the help tag
mumi command -t -moreinfo -t +help