Skip to contentSkip to conversation

manpage: update mount.cifs manpage with info aboutincomplete options

PATCH Jeff Layton 1 participant 1 message Jan 2, 2018 2 messages not in the archive

Subsystems (2)
lore
Share & tools
lore

A message that is not in the archive (sent off-list or not yet received): [email protected]

A message that is not in the archive (sent off-list or not yet received): [email protected]

#2 lore raw

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 options

Quoted 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 a1f3acd40b265f134a97a739a6898b3958d206b9 modified mount parameters, but not updated mount.cifs manpage. Fix it.

---
mount.cifs.rst | 10 +++++-----

1 file changed, 5 insertions(+), 5 deletions(-)
mount.cifs.rst +1 -1
index 023ff34..a4445b8 100644
@@ -48,7 +48,7 @@ OPTIONS
 *******
 
 
-username=arg
+username=arg|user=arg

specifies 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=arg

non-superuser mount. It is therefore recommended to use the full ``username=`` option name.

mount.cifs.rst +1 -1
-password=arg
+password=arg|pass=arg

specifies 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=arg

variable 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=filename

specifies 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=arg

If 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=arg

sets 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=arg

sets 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]>