guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH (4)] gnu: Add NFS related services. (minor improvements to do


From: Ludovic Courtès
Subject: Re: [PATCH (4)] gnu: Add NFS related services. (minor improvements to documentation; Added test to ensure that pipefs mount/umount succeeded()
Date: Tue, 11 Oct 2016 22:30:41 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)

Hi,

John Darrington <address@hidden> skribis:

> Yet another patch.  Here I check that the mount/umount succeeds by testing 
> that 
> /var/lib/nfs/pipefs is a member of the list returned by (mount-points).

OK.

> Regarding Ludo's suggestion to use a filesystem extention a la elogind - I 
> have
> looked into that possibility, but so far as I can see using a service 
> extension 
> does not allow a "provision" field, so I don't know how to make the other 
> services
> dependent upon it.  Also I don't see from the elogind example how to specify 
> an 
> alternative mount point and then to pass that mount point to dependent 
> services.
> (I cannot imagine why anyone would want to do that, but hey - guix is 
> supposed to
> be hackable!)   I'd be happy to change it later if someone can explain how to 
> do it.

Yes, let’s keep it for later.

For future reference, as I wrote in
<https://lists.gnu.org/archive/html/guix-devel/2016-09/msg01096.html>,
the Shepherd service corresponding to the file system object would be
called ‘file-system-/var/lib/nfs/pipefs’.  This is the name you would
write in ‘requirement’.

>
>
>
>
> * gnu/services/nfs.scm (pipefs-service-type): New Variable,
> (gss-service-type): New Variable, (idmap-service-type) New Variable.
>
> * doc/guix.texi (Network File system): New Node.

No need to capitalize “node” and “variable.”

OK with this patch, thank you!

Ludo’.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]