On Tue, 23 Oct 2012 18:47:37 +0200 steve <steve@xxxxxxxxxxxx> wrote: > On 10/23/2012 05:56 PM, Scott Lovenberg wrote: > > On 10/18/2012 2:07 PM, scott.lovenberg@xxxxxxxxx wrote: > > no one has objected (or really said anything). Can we merge this patch? > > -- > Hi > I'm just trying to represent users. Can we take this to user level by > giving an example of what will work and what will not work after the patch? > > For example, the Linux automounter. > > Currently, we have this map: > * -fstype=cifs,rw,sec=krb5 ://myserver/myshare/& > Does that really work? What purpose does the ':' serve there? That should probably be removed. I doubt we'd end up breaking that syntax, but I can't be certain. > Are you talking about the difference between that and this: > * -fstype=cifs,rw,sec=krb5 myserver:/myshare/& Right, the above syntax would no longer work after the change. > > Question: will I need to change anything due to this patch? > For this patch, you don't need to do anything. It just adds a warning. Eventually though, nfs-style "devicenames" would no longer work for cifs mounts. For your map above, you probably want something like: * -fstype=cifs,rw,sec=krb5 //myserver/myshare/& (i.e. get rid of the extraneous ':'). -- Jeff Layton <jlayton@xxxxxxxxxx> -- 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
