Re: [PATCH 1/2] ksmbd: update documentation

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

 



2022-09-02 6:48 GMT+09:00, Jeremy Allison <jra@xxxxxxxxx>:
> On Thu, Sep 01, 2022 at 04:37:21PM -0500, Steve French wrote:
>>
>>I do like that Namjae et al made the format of the .conf file very similar
>>to the format of Samba's smb.conf file though.   I realize there some
>>users complain that there are too many smb.conf parameters for Samba,
>>but he seemed to pick a reasonably subset of them for ksmbd.
>>Samba is a much larger project with many more smb.conf parameters
>>but it does reduce confusion making the parameter names similar
>>where possible e.g. "workgroup", "guest account", (share) path, read only,
>> etc.
>>and fortunately the default directories for the two smb.conf files are
>>different so at least the daemons don't use the same file.
>
> Sure, I think the formats and parameter names being as close
> as possible is a great idea to allow users to move between
> servers as they wish. But making the files the same name
> is not a good idea.
This is the first time I've heard that it is problem that ksmbd's
config filename is same with samba's one. Reading the mail threads, I
don't understand exactly what the real problem is... I thought that
using same smb.conf name make users aware that it was forked from
samba's one. I'm a little surprised, I thought that you will say that
we should use the same name. This seems to contradict your previous
opinion that ksmbd's dos attribute and stream xattr format should be
the same with samba's one. It's not difficult to change config
filename of ksmbd if you agree with it. If we think about adding the
ksmbd integration feature in samba recently, I think it would be
better to use the same name for future.



[Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux