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

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

[Michael Rohleder]

[-- Attachment #1.1: Type: text/plain, Size: 267 bytes --]

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.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: [PATCH] doc: add comment about ovl for Pam mount. --]
[-- Type: text/x-patch, Size: 867 bytes --]

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)")
             (mkmountpoint (@@ (enable "1")
                              (remove "true")))))
 
-- 
2.26.2


[-- Attachment #1.3: Type: text/plain, Size: 287 bytes --]


Footnotes: 
[1]  https://github.com/ghthor/hxtools

-- 
"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]

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]

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

[Ludovic Courtès]

Hi,

This patch fell through the cracks…

Michael Rohleder <mike@rohleder.de> skribis:

> 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’.

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

[Michael Rohleder]

[-- Attachment #1: Type: text/plain, Size: 623 bytes --]

Hi Ludo!

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.
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.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 511 bytes --]

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

[Ludovic Courtès]

Hi Michael,

Michael Rohleder <mike@rohleder.de> skribis:

> 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.

> 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’.