manpage: update mount.cifs manpage with info aboutincomplete options
Subsystems (2)
Yes, we'll be merging it -- thank you!
I'm in the process of handing off cifs-utils maintainership to Pavel, and he's working on getting his account set up correctly. Once that's done, we'll merge this patch soon afterward.
-- Jeff
Quoted 100 lines · On Tue, 2018-01-02 at 18:17 +0800, zhang.xianwei8-Th6q7B73Y6EnDS1+zs4M5A@public…
So, could merge the patch?
------------------原始邮件------------------
发件人: <[email protected]>;
收件人: <[email protected]>; 抄送人:张贤伟10129966; <[email protected]>; 日 期 :2017年12月08日 22:30
主 题 :Re: [PATCH] manpage: update mount.cifs manpage with info aboutincomplete optionsQuoted 1 line
Anyone have thoughts on this?
My reaction is that it is more work to remove the aliases, and not likely to improve usability in this case to remove the alias - so better to leave the aliases in.
Quoted 68 lines · Jeff Layton
Quoted 53 lines · On Fri, 2017-12-08 at 15:11 +0800, zhang.xianwei8-Th6q7B73Y6EnDS1+zs4M5A@public…
This commit
a1f3acd40b265f134a97a739a6898b3958d206b9modified mount parameters, but not updated mount.cifs manpage. Fix it.Signed-off-by Zhang Xianwei <[email protected]>---
mount.cifs.rst| 10 +++++-----1 file changed, 5 insertions(+), 5 deletions(-)mount.cifs.rst +1
-1@@ -48,7 +48,7 @@ OPTIONS ******* -username=arg +username=arg|user=argspecifies the username to connect as. If this is not given, then the environment variable USER is used.
mount.cifs.rst +0
-0@@ -64,7 +64,7 @@ username=argnon-superuser mount. It is therefore recommended to use the full ``username=`` option name.
mount.cifs.rst +1
-1-password=arg +password=arg|pass=argspecifies the CIFS password. If this option is not given then the environment variable PASSWD is used. If the password is not specified directly or indirectly via an argument to mount, mount.cifs will
mount.cifs.rst +0
-0@@ -76,7 +76,7 @@ password=argvariable or via a credentials file (see below) or entered at the password prompt will be read correctly.
mount.cifs.rst +1
-1-credentials=filename +credentials=filename|cred=filenamespecifies a file that contains a username and/or password and optionally the name of the workgroup. The format of the file is::
mount.cifs.rst +0
-0@@ -159,12 +159,12 @@ dir_mode=argIf the server does not support the CIFS Unix extensions this overrides the default mode for directories.
mount.cifs.rst +1
-1-ip=arg +ip=arg|addr=argsets the destination IP address. This option is set automatically if the server name portion of the requested UNC name can be resolved so rarely needs to be specified by the user.
mount.cifs.rst +1
-1-domain=arg +domain=arg|dom=arg|workgroup=argsets the domain (workgroup) of the user.
Quoted 1 line
guest
Honestly, I'd really prefer to move this in the other direction and have the kernel/userland programs stop accepting some of these aliases. They don't add anything to usability and they are a long-term maintenance burden.
We'd probably need a 2-3 releases where we accept the aliases and throw a warning when someone tries to use them. Then we can just rip out support for them.
That said, if the consensus is to keep the aliases in, then I'll just plan to merge this patch.
Quoted 7 lines
Anyone have thoughts on this?
--
Jeff Layton <[email protected]> --
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html--
Thanks,Steve
Signature · 1 line
Jeff Layton <[email protected]>