Re: CIFS: Deprecating NFS mounting syntax in mount.cifs

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

 



On Tue, Oct 23, 2012 at 3:23 PM, steve <steve@xxxxxxxxxxxx> wrote:

> Hi Scott, hi everyone
> Yeah, that's fine.
> Does this clear up the issue with the ':'? I should have made it clearer
> that I was referring to autofs and not mounting e.g. from fstab. I just
> tried the automounter on cifs without the ':' and it doesn't work.
>
> Would it perhaps help to put a message in the logs when it fails, rather
> than silence? Or maybe that's more of a question for the autofs guys.
> Cheers,
> Steve
>

I've been at home thinking about this for a while tonight. I've
checked the documentation for autofs and they do what they say what
they'll do with that path (treat anything without a ':' as an NFS
mount).  On our side, (mount.cifs) we do what we say we'll do (support
UNC paths).  The most we could ask of them is to add/modify their
documentation to include the case for CIFS instead of just SMB.

This doesn't change anything on the mount.cifs side other than
explicitly directing users to the correct syntax for CIFS shares when
using autofs.  Ultimately the autofs documentation implicitly states
that CIFS shares should use a ':'.

All that being said, the mount.cifs has never officially supported NFS
path syntax.  We aren't silently ignoring the issue; we're sending a
warning to stdout that in a future version of the mount utility we
won't support this undocumented behavior.  To be fair, that's more
than most code bases do for deprecating undocumented features.

If anyone wants me to pursue the issue, I'll see what I can do about
getting the documentation for autofs altered to explicitly mention
CIFS paths.  I think that is reasonable for everyone.  It's after 2 AM
in my part of the world, so I'll do this tomorrow after my first cup
of coffee if anyone requests it.

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


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

  Powered by Linux