Re: [PATCH] nfs.man: add description of multiple sec= options

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

 



On Oct 29, 2013, at 12:36 PM, Myklebust, Trond <Trond.Myklebust@xxxxxxxxxx> wrote:

>> -----Original Message-----
>> From: linux-nfs-owner@xxxxxxxxxxxxxxx [mailto:linux-nfs-
>> owner@xxxxxxxxxxxxxxx] On Behalf Of Chuck Lever
>> Sent: Tuesday, October 29, 2013 12:30 PM
>> To: Weston Andros Adamson
>> Cc: steved@xxxxxxxxxx; linux-nfs@xxxxxxxxxxxxxxx
>> Subject: Re: [PATCH] nfs.man: add description of multiple sec= options
>> 
>> 
>> On Oct 29, 2013, at 12:27 PM, Weston Andros Adamson <dros@xxxxxxxxxx>
>> wrote:
>> 
>>> The client now supports multiple sec= options as a colon delimited list.
>>> 
>>> Signed-off-by: Weston Andros Adamson <dros@xxxxxxxxxx>
>>> ---
>>> utils/mount/nfs.man | 7 ++++---
>>> 1 file changed, 4 insertions(+), 3 deletions(-)
>>> 
>>> diff --git a/utils/mount/nfs.man b/utils/mount/nfs.man index
>>> 2a42b93..17b8d88 100644
>>> --- a/utils/mount/nfs.man
>>> +++ b/utils/mount/nfs.man
>>> @@ -380,9 +380,10 @@ If a value of zero is specified, the .BR mount
>>> (8) command exits immediately after the first failure.
>>> .TP 1.5i
>>> -.BI sec= flavor
>>> -The security flavor to use for accessing files on this mount point.
>>> -If the server does not support this flavor, the mount operation fails.
>>> +.BI sec= flavors
>>> +A colon-delimited list of security flavors to use for accessing files
>>> +on this mount point. If the server does not support any of these
>>> +flavors, the mount operation fails.
>> 
>> Just a nit:  The new text kind of suggests that the colons are required.
>> "sec=single flavor" is also still supported.  Typically man page language is
>> careful to show both.
> 
> How about "colon-separated list of one or more security flavours"? That's less ambiguous than "colon-delimited"…

OK, but fwiw I aped that from the exports manpage:

       sec=   The  sec= option, followed by a colon-delimited list of security
              flavors, restricts the export to clients  using  those  flavors.
              Available  security flavors include sys (the default--no crypto‐
…

So it:
 1) isn’t clear that one flavor is an option.
 2) says “colon-delimited"

Should we clean this up too?

-dros��.n��������+%������w��{.n�����{��w���jg��������ݢj����G�������j:+v���w�m������w�������h�����٥





[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux