Re: [PATCH] nfs-utils: add nfs.idmap

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

 




On 10/28/2010 11:40 AM, Trond Myklebust wrote:
> On Thu, 2010-10-28 at 11:27 -0400, Bryan Schumaker wrote:
>> On 10/28/2010 11:18 AM, Steve Dickson wrote:
>>> Hey Bryan,
>>>
>>> A couple nits.... which I will be more than willing to take of... 
>>>
>>> 1) I would like to change the binary name from nfs.idmap to nfsidmap
>>>    which is consistent with how the rest of the binaries are named.
>>>    (i.e. binary names generally don't have any '.' or '-' in them)
>>>    
>>> 2) Change the source directory from utils/nfs.idmap to 
>>>    utils/idmap which again makes it consistent with what
>>>    is already there. 
>>
>> These both sounds reasonable to me.  I'll make a patch to modify the kernel documentation to reflect this.
>>>
> 
> We just sent in one patch to change the name in the kernel docs, now the
> name is changing again?
I'll take the blame for that... I should have gotten into the 
conversation  earlier... sorry about that...

> 
> Let's wait with this until an actual nfs-utils release is available, so
> we can also update the 'Documentation/Changes' file with the new minimal
> release version number.
Just to be clear... the kernel changes will be in the 2.6.37 kernel, correct?

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