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

Maybe Dros could also update the EXAMPLES section with one of each.  Just a thought.

-- 
Chuck Lever
chuck[dot]lever[at]oracle[dot]com




--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[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