GNU bug report logs - #5345
Password asked when visiting a file in a lightweight checkout

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 5345 in the body.
You can then email your comments to 5345 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Juanma Barranquero <lekktu <at> gmail.com>
To: Bug-Gnu-Emacs <bug-gnu-emacs <at> gnu.org>
Subject: Password asked when visiting a file in a lightweight checkout
Date: Sat, 9 Jan 2010 03:41:28 +0100

With an up-to-date trunk build.

  emacs -Q
  C-x C-f C:/emacs/repo/test-lightweight/etc/NEWS <RET>
  => "Password for Juanma <at> C:"     -- Juanma being my user-name
  C-g
  C-x C-f C:/emacs/repo/test-lightweight/etc/NEWS <RET>
  => the file is visited OK.

It's happening for all files in lightweight checkouts:

  C:\emacs\repo\test-lightweight> bzr info
  Lightweight checkout (format: 2a or development-subtree)
  Location:
         light checkout root: .
    repository checkout root: C:/emacs/repo/trunk
          checkout of branch: sftp://bzr.savannah.gnu.org/srv/bzr/emacs/trunk/
           shared repository: C:/emacs/repo

  Related branches:
    public branch: sftp://bzr.savannah.gnu.org/srv/bzr/emacs/trunk
    parent branch: sftp://bzr.savannah.gnu.org/srv/bzr/emacs/trunk/

It does not happen for heavyweight checkouts, bound branches, or
normal (non-bound, non-checkout) branches.

    Juanma

Message #8 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Michael Albinus <michael.albinus <at> gmx.de>
To: Juanma Barranquero <lekktu <at> gmail.com>
Cc: 5345 <at> debbugs.gnu.org
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight
	checkout
Date: Sat, 09 Jan 2010 11:11:12 +0100

Juanma Barranquero <lekktu <at> gmail.com> writes:

> With an up-to-date trunk build.
>
>   emacs -Q
>   C-x C-f C:/emacs/repo/test-lightweight/etc/NEWS <RET>
>   => "Password for Juanma <at> C:"     -- Juanma being my user-name
>   C-g
>   C-x C-f C:/emacs/repo/test-lightweight/etc/NEWS <RET>
>   => the file is visited OK.

Could you, please, set `debug-on-quit' to t? Then the backtrace after
C-g might be interesting.

>     Juanma

Best regards, Michael.

Message #11 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Juanma Barranquero <lekktu <at> gmail.com>
To: Michael Albinus <michael.albinus <at> gmx.de>
Cc: 5345 <at> debbugs.gnu.org
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight 
	checkout
Date: Sat, 9 Jan 2010 13:44:02 +0100

On Sat, Jan 9, 2010 at 11:11, Michael Albinus <michael.albinus <at> gmx.de> wrote:

> Could you, please, set `debug-on-quit' to t? Then the backtrace after
> C-g might be interesting.

Yes, of course.

The backtrace corresponds to find-file etc/NEWS in branch
C:/emacs/repo/test-lightweight, which is a lightweight checkout of
C:/emacs/repo/trunk.

    Juanma



Debugger entered--Lisp error: (quit)
  byte-code("\301^P\302\303!\207" [quit-flag t eval (ignore nil)] 2)
  read-passwd("Password for Juanma <at> C: ")
  ange-ftp-get-passwd("C" "Juanma")
  ange-ftp-get-process("C" "Juanma")
  ange-ftp-host-type("C" "Juanma")
  ange-ftp-ls("/C:/emacs/repo/trunk/.bzr/branch/format/" "-al" t t)
  ange-ftp-get-files("/C:/emacs/repo/trunk/.bzr/branch/format" t)
  ange-ftp-file-entry-p("/C:/emacs/repo/trunk/.bzr/branch/format")
  ange-ftp-file-exists-p("/C:/emacs/repo/trunk/.bzr/branch/format")
  apply(ange-ftp-file-exists-p "/C:/emacs/repo/trunk/.bzr/branch/format")
  byte-code("\304^X\305 ^Y\306\216\307\n^K\"+\207" [debug-on-error
save-match-data-internal fn args t match-data ((byte-code
"\301^H\302\"\207" [save-match-data-internal set-match-data evaporate]
3)) apply] 3)
  ange-ftp-hook-function(file-exists-p
"/C:/emacs/repo/trunk/.bzr/branch/format")
  apply(ange-ftp-hook-function file-exists-p
"/C:/emacs/repo/trunk/.bzr/branch/format")
  tramp-ftp-file-name-handler(file-exists-p
"/C:/emacs/repo/trunk/.bzr/branch/format")
  apply(tramp-ftp-file-name-handler file-exists-p
"/C:/emacs/repo/trunk/.bzr/branch/format")
  tramp-file-name-handler(file-exists-p
"/C:/emacs/repo/trunk/.bzr/branch/format")
  file-exists-p("/C:/emacs/repo/trunk/.bzr/branch/format")
  vc-bzr-working-revision("c:/emacs/repo/test-lightweight/etc/NEWS")
  apply(vc-bzr-working-revision "c:/emacs/repo/test-lightweight/etc/NEWS")
  vc-call-backend(Bzr working-revision
"c:/emacs/repo/test-lightweight/etc/NEWS")
  vc-working-revision("c:/emacs/repo/test-lightweight/etc/NEWS" Bzr)
  vc-default-mode-line-string(Bzr "c:/emacs/repo/test-lightweight/etc/NEWS")
  apply(vc-default-mode-line-string Bzr
"c:/emacs/repo/test-lightweight/etc/NEWS")
  vc-call-backend(Bzr mode-line-string
"c:/emacs/repo/test-lightweight/etc/NEWS")
  vc-mode-line("c:/emacs/repo/test-lightweight/etc/NEWS" Bzr)
  vc-find-file-hook()
  run-hooks(find-file-hook)
  after-find-file(nil t)
  find-file-noselect-1(#<buffer NEWS>
"c:/emacs/repo/test-lightweight/etc/NEWS" nil nil
"c:/emacs/repo/test-lightweight/etc/NEWS" ((3072 4 . 28960) (10917 .
23791)))
  find-file-noselect("c:/emacs/repo/test-lightweight/etc/NEWS" nil nil t)
  find-file("c:/emacs/repo/test-lightweight/etc/NEWS" t)
  call-interactively(find-file nil nil)

Message #14 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Juanma Barranquero <lekktu <at> gmail.com>, 5345 <at> debbugs.gnu.org
Subject: Re: bug#5345: Password asked when visiting a file in a
	lightweight	checkout
Date: Sat, 09 Jan 2010 15:52:33 +0200

> From: Juanma Barranquero <lekktu <at> gmail.com>
> Date: Sat, 9 Jan 2010 13:44:02 +0100
> Cc: 5345 <at> debbugs.gnu.org
> 
>   file-exists-p("/C:/emacs/repo/trunk/.bzr/branch/format")  <<<<<<<<<<
>   vc-bzr-working-revision("c:/emacs/repo/test-lightweight/etc/NEWS")
>   apply(vc-bzr-working-revision "c:/emacs/repo/test-lightweight/etc/NEWS")
>   vc-call-backend(Bzr working-revision "c:/emacs/repo/test-lightweight/etc/NEWS")

Here's the bug: some code thinks that "C:/foo" is not an absolute file
name, probably because it checks for the leading slash.  So it
prepends a slash, which alters the semantics of the file to be a
remote file on a mythical machine called "C", with predictable
results.

I'd take a good look at vc-bzr-working-revision and the subroutines it
calls.

Message #17 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Juanma Barranquero <lekktu <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: Dan Nicolaescu <dann <at> ics.uci.edu>, 5345 <at> debbugs.gnu.org
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight 
	checkout
Date: Sat, 9 Jan 2010 15:36:29 +0100

On Sat, Jan 9, 2010 at 14:52, Eli Zaretskii <eliz <at> gnu.org> wrote:

> Here's the bug:

You're right.

> some code thinks that "C:/foo" is not an absolute file
> name, probably because it checks for the leading slash.

In fact, what happens is that a regular expression is removing the
initial file:// from a file: URI, and not taking into account that it
could start with a triple-slash. Or it *is* taking it into account and
doing it on purpose to get a /path on Unix.

If removing three slashes is OK, the patch is as easy as changing
"file://\\(.+\\)" to "file:///?\\(.+\\)". If not, a check for windows
(or for a Windows-style path) will have to be done.

Dan, comments?

   Juanma

Message #20 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Dan Nicolaescu <dann <at> ics.uci.edu>
To: Juanma Barranquero <lekktu <at> gmail.com>
Cc: 5345 <at> debbugs.gnu.org
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight
	checkout
Date: Sat, 9 Jan 2010 10:46:57 -0800 (PST)

Juanma Barranquero <lekktu <at> gmail.com> writes:

  > On Sat, Jan 9, 2010 at 14:52, Eli Zaretskii <eliz <at> gnu.org> wrote:
  > 
  > > Here's the bug:
  > 
  > You're right.
  > 
  > > some code thinks that "C:/foo" is not an absolute file
  > > name, probably because it checks for the leading slash.
  > 
  > In fact, what happens is that a regular expression is removing the
  > initial file:// from a file: URI, and not taking into account that it
  > could start with a triple-slash. Or it *is* taking it into account and
  > doing it on purpose to get a /path on Unix.
  > 
  > If removing three slashes is OK, the patch is as easy as changing

The third slash is part of the absolute file name, so it needs to stay.

Message #23 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Michael Albinus <michael.albinus <at> gmx.de>
To: Juanma Barranquero <lekktu <at> gmail.com>
Cc: 5345 <at> debbugs.gnu.org
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight
	checkout
Date: Sat, 09 Jan 2010 19:53:08 +0100

Juanma Barranquero <lekktu <at> gmail.com> writes:

>   tramp-file-name-handler(file-exists-p
> "/C:/emacs/repo/trunk/.bzr/branch/format")
>   file-exists-p("/C:/emacs/repo/trunk/.bzr/branch/format")
>   vc-bzr-working-revision("c:/emacs/repo/test-lightweight/etc/NEWS")

Adding a leading "/" to "C:/emacs/repo/trunk/.bzr/branch/format" in
`vc-bzr-working-revision' seems to be the culprit.

Best regards, Michael.

Message #26 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Juanma Barranquero <lekktu <at> gmail.com>
To: Dan Nicolaescu <dann <at> ics.uci.edu>
Cc: 5345 <at> debbugs.gnu.org
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight 
	checkout
Date: Sat, 9 Jan 2010 20:06:49 +0100

On Sat, Jan 9, 2010 at 19:46, Dan Nicolaescu <dann <at> ics.uci.edu> wrote:

> The third slash is part of the absolute file name

Not for Windows file:// URIs. For example, in one of my lightweight
checkouts, .bzr/branch/location contains:

  file:///C:/emacs/repo/bugs/5313/

> so it needs to stay.

OK, then we must also check whether the absolute filename is
Windows-style, i.e. ^/?[A-Z]:.*$ (assuming the unwanted first slash).
Do you want to fix it?

    Juanma

Message #29 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Dan Nicolaescu <dann <at> ics.uci.edu>
To: Juanma Barranquero <lekktu <at> gmail.com>
Cc: 5345 <at> debbugs.gnu.org
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight
	checkout
Date: Sat, 9 Jan 2010 11:20:52 -0800 (PST)

Juanma Barranquero <lekktu <at> gmail.com> writes:

  > On Sat, Jan 9, 2010 at 19:46, Dan Nicolaescu <dann <at> ics.uci.edu> wrote:
  > 
  > > The third slash is part of the absolute file name
  > 
  > Not for Windows file:// URIs. For example, in one of my lightweight

I meant for unix-like systems.

  > checkouts, .bzr/branch/location contains:
  > 
  >   file:///C:/emacs/repo/bugs/5313/

Is that the correct URL syntax?  If not, they please let the bzr people
know about it.

  > > so it needs to stay.
  > 
  > OK, then we must also check whether the absolute filename is
  > Windows-style, i.e. ^/?[A-Z]:.*$ (assuming the unwanted first slash).
  > Do you want to fix it?

No.  I can't test it.
But please do it with a system-type test, /C:/emacs/repo/bugs/5313/ is a
valid unix file name.  Probably not used too much, but valid, so it
should not be excluded.

Message #32 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Juanma Barranquero <lekktu <at> gmail.com>
To: Dan Nicolaescu <dann <at> ics.uci.edu>
Cc: 5345 <at> debbugs.gnu.org
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight 
	checkout
Date: Sat, 9 Jan 2010 21:08:58 +0100

On Sat, Jan 9, 2010 at 20:20, Dan Nicolaescu <dann <at> ics.uci.edu> wrote:

> Is that the correct URL syntax?

Yes. This kind of URL is very common when dealing with files on
Windows. According to RFC 1738 ("Uniform Resource Locators (URL)"),
section 3.10 ("FILES"), the syntax is

     file://<host>/<path>

i.e., the slashes are separators. And "[a]s a special case, <host> can
be the string "localhost" or the empty string; this is interpreted as
`the machine from which the URL is being interpreted'.". So
file:///C:/path is a perfectly valid URL for a local file on Windows,
as a shorthand for file://localhost/C:/path. I'm surprised Unix URLs
for absolute paths do not start with file:////.

> But please do it with a system-type test, /C:/emacs/repo/bugs/5313/ is a
> valid unix file name.  Probably not used too much, but valid, so it
> should not be excluded.

Are you OK with the following patch?

    Juanma


=== modified file 'lisp/vc-bzr.el'
--- lisp/vc-bzr.el	2010-01-06 15:11:52 +0000
+++ lisp/vc-bzr.el	2010-01-09 20:06:10 +0000
@@ -361,6 +361,11 @@
 		     ;; look there for the version information.
 		     (when (re-search-forward "file://\\(.+\\)" nil t)
 		       (let ((l-c-parent-dir (match-string 1)))
+			 (when (and (memq system-type '(ms-dos windows-nt))
+				    (string-match-p "^/[[:alpha:]]:" l-c-parent-dir))
+			   ;;; On Windows, file:// URLs often have three slashes,
+			   ;;; so we must remove the remaining one (bug#5345)
+			   (setq l-c-parent-dir (substring l-c-parent-dir 1)))
 			 (setq branch-format-file
 			       (expand-file-name vc-bzr-admin-branch-format-file
 						 l-c-parent-dir))

Message #35 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Dan Nicolaescu <dann <at> ics.uci.edu>
To: Juanma Barranquero <lekktu <at> gmail.com>
Cc: 5345 <at> debbugs.gnu.org
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight
	checkout
Date: Sat, 9 Jan 2010 12:39:02 -0800 (PST)

Juanma Barranquero <lekktu <at> gmail.com> writes:

  > On Sat, Jan 9, 2010 at 20:20, Dan Nicolaescu <dann <at> ics.uci.edu> wrote:
  > 
  > > Is that the correct URL syntax?
  > 
  > Yes. This kind of URL is very common when dealing with files on
  > Windows. According to RFC 1738 ("Uniform Resource Locators (URL)"),
  > section 3.10 ("FILES"), the syntax is
  > 
  >      file://<host>/<path>
  > 
  > i.e., the slashes are separators. And "[a]s a special case, <host> can
  > be the string "localhost" or the empty string; this is interpreted as
  > `the machine from which the URL is being interpreted'.". So
  > file:///C:/path is a perfectly valid URL for a local file on Windows,
  > as a shorthand for file://localhost/C:/path. I'm surprised Unix URLs
  > for absolute paths do not start with file:////.
  > 
  > > But please do it with a system-type test, /C:/emacs/repo/bugs/5313/ is a
  > > valid unix file name.  Probably not used too much, but valid, so it
  > > should not be excluded.
  > 
  > Are you OK with the following patch?

Sure.

Message #38 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Lennart Borgman <lennart.borgman <at> gmail.com>
To: Dan Nicolaescu <dann <at> ics.uci.edu>, 5345 <at> debbugs.gnu.org
Cc: Juanma Barranquero <lekktu <at> gmail.com>
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight 
	checkout
Date: Sat, 9 Jan 2010 21:57:58 +0100

On Sat, Jan 9, 2010 at 9:39 PM, Dan Nicolaescu <dann <at> ics.uci.edu> wrote:
> Juanma Barranquero <lekktu <at> gmail.com> writes:
>
>  > On Sat, Jan 9, 2010 at 20:20, Dan Nicolaescu <dann <at> ics.uci.edu> wrote:
>  >
>  > > Is that the correct URL syntax?
>  >
>  > Yes. This kind of URL is very common when dealing with files on
>  > Windows. According to RFC 1738 ("Uniform Resource Locators (URL)"),
>  > section 3.10 ("FILES"), the syntax is
>  >
>  >      file://<host>/<path>
>  >
>  > i.e., the slashes are separators. And "[a]s a special case, <host> can
>  > be the string "localhost" or the empty string; this is interpreted as
>  > `the machine from which the URL is being interpreted'.". So
>  > file:///C:/path is a perfectly valid URL for a local file on Windows,
>  > as a shorthand for file://localhost/C:/path. I'm surprised Unix URLs
>  > for absolute paths do not start with file:////.


We were discussing before if file://c:/some/where.txt also was
correct. We could not rule out the possibility that it was. So please
take care of this case too.


>  > > But please do it with a system-type test, /C:/emacs/repo/bugs/5313/ is a
>  > > valid unix file name.  Probably not used too much, but valid, so it
>  > > should not be excluded.
>  >
>  > Are you OK with the following patch?
>
> Sure.

Message #41 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Juanma Barranquero <lekktu <at> gmail.com>
To: Lennart Borgman <lennart.borgman <at> gmail.com>
Cc: 5345 <at> debbugs.gnu.org, Dan Nicolaescu <dann <at> ics.uci.edu>
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight 
	checkout
Date: Sat, 9 Jan 2010 22:04:13 +0100

On Sat, Jan 9, 2010 at 21:57, Lennart Borgman <lennart.borgman <at> gmail.com> wrote:

> We were discussing before if file://c:/some/where.txt also was
> correct. We could not rule out the possibility that it was.

I don't think it is. RFC 1378 allows file://host/path, and some other
RFC (don't remember the #) apparently allows file:/path, but that
wouldn't grok file://c:/my/path (though it could grok file://my/path,
if the drive is omitted).

> So please take care of this case too.

It's already taken care of. Look at the patch.

    Juanma

Message #46 received at 5345-done <at> debbugs.gnu.org (full text, mbox):

From: Juanma Barranquero <lekktu <at> gmail.com>
To: 5345-done <at> debbugs.gnu.org
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight 
	checkout
Date: Sat, 9 Jan 2010 22:08:04 +0100

Patch installed.

Message #49 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Lennart Borgman <lennart.borgman <at> gmail.com>
To: Juanma Barranquero <lekktu <at> gmail.com>
Cc: 5345 <at> debbugs.gnu.org, Dan Nicolaescu <dann <at> ics.uci.edu>
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight 
	checkout
Date: Sat, 9 Jan 2010 22:12:19 +0100

On Sat, Jan 9, 2010 at 10:04 PM, Juanma Barranquero <lekktu <at> gmail.com> wrote:
> On Sat, Jan 9, 2010 at 21:57, Lennart Borgman <lennart.borgman <at> gmail.com> wrote:
>
>> We were discussing before if file://c:/some/where.txt also was
>> correct. We could not rule out the possibility that it was.
>
> I don't think it is. RFC 1378 allows file://host/path, and some other
> RFC (don't remember the #) apparently allows file:/path,


I am surprised, but I do not remember the details any more.


> but that
> wouldn't grok file://c:/my/path


... on un*x? ... ;-)

Maybe this is where the confusion comes from (but the spec seemed
unclear to me).


(though it could grok file://my/path,
> if the drive is omitted).
>
>> So please take care of this case too.
>
> It's already taken care of. Look at the patch.


Thanks.


>    Juanma
>

Message #52 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Michael Albinus <michael.albinus <at> gmx.de>
To: Dan Nicolaescu <dann <at> ics.uci.edu>
Cc: 5345 <at> debbugs.gnu.org
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight
	checkout
Date: Sat, 09 Jan 2010 22:15:10 +0100

Dan Nicolaescu <dann <at> ics.uci.edu> writes:

> But please do it with a system-type test, /C:/emacs/repo/bugs/5313/ is a
> valid unix file name.  Probably not used too much, but valid, so it
> should not be excluded.

Emacs would handle it as a remote file name. If such a local file name
is intended, it must be prefixed by "/:". Maybe it is a good idea to do
it for all URLs starting with "file:///" in vc-bzr (and somewhere else).

Best regards, Michael.

Message #55 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Dan Nicolaescu <dann <at> ics.uci.edu>
To: Michael Albinus <michael.albinus <at> gmx.de>
Cc: 5345 <at> debbugs.gnu.org
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight
	checkout
Date: Sat, 9 Jan 2010 13:33:10 -0800 (PST)

Michael Albinus <michael.albinus <at> gmx.de> writes:

  > Dan Nicolaescu <dann <at> ics.uci.edu> writes:
  > 
  > > But please do it with a system-type test, /C:/emacs/repo/bugs/5313/ is a
  > > valid unix file name.  Probably not used too much, but valid, so it
  > > should not be excluded.
  > 
  > Emacs would handle it as a remote file name. If such a local file name
  > is intended, it must be prefixed by "/:". Maybe it is a good idea to do
  > it for all URLs starting with "file:///" in vc-bzr (and somewhere else).

Can handling as a remote file name be disabled?
The code path in question makes sense when the file is local, if the
file is remote, then just running the corresponding bzr command won't
make a difference.

Message #58 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Michael Albinus <michael.albinus <at> gmx.de>
To: Dan Nicolaescu <dann <at> ics.uci.edu>
Cc: 5345 <at> debbugs.gnu.org
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight
	checkout
Date: Sun, 10 Jan 2010 07:51:44 +0100

Dan Nicolaescu <dann <at> ics.uci.edu> writes:

> Can handling as a remote file name be disabled?
> The code path in question makes sense when the file is local, if the
> file is remote, then just running the corresponding bzr command won't
> make a difference.

Adding "/:" at the top of the file name is one way. Another (maybe more
common) is let-binding `file-name-handler-alist' to nil.

OTOH, it might be that vc-bzr is handling a remote file. Directory names
found in (remote) control files must be regarded remote as well. If they
are just being used as arguments of processes, there is no problem. But
if they are used in (for example) `file-exists-p', the file name must be
extended by the remote component.

Best regards, Michael.

Message #61 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Reiner Steib <reinersteib+gmane <at> imap.cc>
To: Juanma Barranquero <lekktu <at> gmail.com>
Cc: 5345 <at> debbugs.gnu.org
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight
	checkout
Date: Sun, 10 Jan 2010 15:10:50 +0100

On Sat, Jan 09 2010, Juanma Barranquero wrote:

> According to RFC 1738 ("Uniform Resource Locators (URL)"),
> section 3.10 ("FILES"), the syntax is
>
>      file://<host>/<path>
>
> i.e., the slashes are separators. And "[a]s a special case, <host> can
> be the string "localhost" or the empty string; this is interpreted as
> `the machine from which the URL is being interpreted'.". So
> file:///C:/path is a perfectly valid URL for a local file on Windows,
> as a shorthand for file://localhost/C:/path. I'm surprised Unix URLs
> for absolute paths do not start with file:////.

I think the RFC assumes that the leading "/" is not part of the
directory name, so file://<host>/<path> with <host> = localhost and
<path> = /etc/fstab (etc/fstab relative to /) becomes
file:///etc/fstab just like /pub/README on ftp.gnu.org is written as
ftp://ftp.gnu.org/pub/README and not ftp://ftp.gnu.org//pub/README

,----[ rfc1738 ]
| 3.10 FILES
| 
|    The file URL scheme is used to designate files accessible on a
|    particular host computer. This scheme, unlike most other URL schemes,
|    does not designate a resource that is universally accessible over the
|    Internet.
| 
|    A file URL takes the form:
| 
|        file://<host>/<path>
| 
|    where <host> is the fully qualified domain name of the system on
|    which the <path> is accessible, and <path> is a hierarchical
|    directory path of the form <directory>/<directory>/.../<name>.
`----

Bye, Reiner.
-- 
       ,,,
      (o o)
---ooO-(_)-Ooo---  |  PGP key available  |  http://rsteib.home.pages.de/

Message #64 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Lennart Borgman <lennart.borgman <at> gmail.com>
To: Reiner Steib <Reiner.Steib <at> gmx.de>, 5345 <at> debbugs.gnu.org, 
	Juanma Barranquero <lekktu <at> gmail.com>
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight 
	checkout
Date: Sun, 10 Jan 2010 18:48:15 +0100

On Sun, Jan 10, 2010 at 3:10 PM, Reiner Steib <reinersteib+gmane <at> imap.cc> wrote:
> On Sat, Jan 09 2010, Juanma Barranquero wrote:
>
>> According to RFC 1738 ("Uniform Resource Locators (URL)"),
>> section 3.10 ("FILES"), the syntax is
>>
>>      file://<host>/<path>
>>
>> i.e., the slashes are separators. And "[a]s a special case, <host> can
>> be the string "localhost" or the empty string; this is interpreted as
>> `the machine from which the URL is being interpreted'.". So
>> file:///C:/path is a perfectly valid URL for a local file on Windows,
>> as a shorthand for file://localhost/C:/path. I'm surprised Unix URLs
>> for absolute paths do not start with file:////.
>
> I think the RFC assumes that the leading "/" is not part of the
> directory name, so file://<host>/<path> with <host> = localhost and
> <path> = /etc/fstab (etc/fstab relative to /) becomes
> file:///etc/fstab just like /pub/README on ftp.gnu.org is written as
> ftp://ftp.gnu.org/pub/README and not ftp://ftp.gnu.org//pub/README


That sounds like a plausible reading to me. Shouldn't this then mean
that file://c:/some/file.txt is correct, but that
file:///c:/bad/example.txt is wrong?


> ,----[ rfc1738 ]
> | 3.10 FILES
> |
> |    The file URL scheme is used to designate files accessible on a
> |    particular host computer. This scheme, unlike most other URL schemes,
> |    does not designate a resource that is universally accessible over the
> |    Internet.
> |
> |    A file URL takes the form:
> |
> |        file://<host>/<path>
> |
> |    where <host> is the fully qualified domain name of the system on
> |    which the <path> is accessible, and <path> is a hierarchical
> |    directory path of the form <directory>/<directory>/.../<name>.
> `----
>
> Bye, Reiner.
> --
>       ,,,
>      (o o)
> ---ooO-(_)-Ooo---  |  PGP key available  |  http://rsteib.home.pages.de/
>
>
>
>
>

Message #67 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Michael Albinus <michael.albinus <at> gmx.de>, 5345 <at> debbugs.gnu.org
Subject: Re: bug#5345: Password asked when visiting a file in a
	lightweight	checkout
Date: Sun, 10 Jan 2010 19:52:12 +0200

> From: Michael Albinus <michael.albinus <at> gmx.de>
> Date: Sun, 10 Jan 2010 07:51:44 +0100
> Cc: 5345 <at> debbugs.gnu.org
> 
> Dan Nicolaescu <dann <at> ics.uci.edu> writes:
> 
> > Can handling as a remote file name be disabled?
> > The code path in question makes sense when the file is local, if the
> > file is remote, then just running the corresponding bzr command won't
> > make a difference.
> 
> Adding "/:" at the top of the file name is one way. Another (maybe more
> common) is let-binding `file-name-handler-alist' to nil.

But the latter is too gross, isn't it?  `file-name-handler-alist'
could hold anything, not just handlers for remote files.

Message #70 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Michael Albinus <michael.albinus <at> gmx.de>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 5345 <at> debbugs.gnu.org
Subject: Re: bug#5345: Password asked when visiting a file in a
	lightweight	checkout
Date: Sun, 10 Jan 2010 20:07:37 +0100

Eli Zaretskii <eliz <at> gnu.org> writes:

>> > Can handling as a remote file name be disabled?
>> > The code path in question makes sense when the file is local, if the
>> > file is remote, then just running the corresponding bzr command won't
>> > make a difference.
>> 
>> Adding "/:" at the top of the file name is one way. Another (maybe more
>> common) is let-binding `file-name-handler-alist' to nil.
>
> But the latter is too gross, isn't it?  `file-name-handler-alist'
> could hold anything, not just handlers for remote files.

You do it only when you are sure it does not hurt. Wrapping a call to
`file-exists-p', for example.

Best regards, Michael.

Message #73 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Juanma Barranquero <lekktu <at> gmail.com>
To: Lennart Borgman <lennart.borgman <at> gmail.com>
Cc: 5345 <at> debbugs.gnu.org, Reiner Steib <Reiner.Steib <at> gmx.de>
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight 
	checkout
Date: Sun, 10 Jan 2010 20:27:59 +0100

On Sun, Jan 10, 2010 at 18:48, Lennart Borgman
<lennart.borgman <at> gmail.com> wrote:

> That sounds like a plausible reading to me. Shouldn't this then mean
> that file://c:/some/file.txt is correct, but that
> file:///c:/bad/example.txt is wrong?

Reading that how, exactly?

    Juanma

Message #76 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Lennart Borgman <lennart.borgman <at> gmail.com>
To: Juanma Barranquero <lekktu <at> gmail.com>
Cc: 5345 <at> debbugs.gnu.org, Reiner Steib <Reiner.Steib <at> gmx.de>
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight 
	checkout
Date: Sun, 10 Jan 2010 20:40:46 +0100

On Sun, Jan 10, 2010 at 8:27 PM, Juanma Barranquero <lekktu <at> gmail.com> wrote:
> On Sun, Jan 10, 2010 at 18:48, Lennart Borgman
> <lennart.borgman <at> gmail.com> wrote:
>
>> That sounds like a plausible reading to me. Shouldn't this then mean
>> that file://c:/some/file.txt is correct, but that
>> file:///c:/bad/example.txt is wrong?
>
> Reading that how, exactly?


It is more like a sound ;-)

Reiner whispered that <host> was collapsed to nothing when the url was
local, ie a local file here. Then we get file:///some/file.txt on
un*xes, but on w32 we get the first case above. There is no way to get
the second case on w32 if Reiners suggestion for reading is correct
(and I am not mistaken...)

Message #79 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Juanma Barranquero <lekktu <at> gmail.com>
To: Lennart Borgman <lennart.borgman <at> gmail.com>
Cc: 5345 <at> debbugs.gnu.org, Reiner Steib <Reiner.Steib <at> gmx.de>
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight 
	checkout
Date: Sun, 10 Jan 2010 21:06:31 +0100

> Reiner whispered that <host> was collapsed to nothing when the url was
> local, ie a local file here.

No, he didn't "whisper" nothing like that. He said that he thinks that
the RFC assumes that the path does not start with a slash.

> There is no way to get
> the second case on w32 if Reiners suggestion for reading is correct
> (and I am not mistaken...)

I think you are.

    Juanma

Message #82 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Lennart Borgman <lennart.borgman <at> gmail.com>
To: Juanma Barranquero <lekktu <at> gmail.com>
Cc: 5345 <at> debbugs.gnu.org, Reiner Steib <Reiner.Steib <at> gmx.de>
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight 
	checkout
Date: Sun, 10 Jan 2010 21:10:28 +0100

On Sun, Jan 10, 2010 at 9:06 PM, Juanma Barranquero <lekktu <at> gmail.com> wrote:
>> Reiner whispered that <host> was collapsed to nothing when the url was
>> local, ie a local file here.
>
> No, he didn't "whisper" nothing like that.

Hm. Are you sure?

> He said that he thinks that
> the RFC assumes that the path does not start with a slash.

Yes.

>> There is no way to get
>> the second case on w32 if Reiners suggestion for reading is correct
>> (and I am not mistaken...)
>
> I think you are.

Please explain. How do you get file:/// on w32?

Message #85 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Juanma Barranquero <lekktu <at> gmail.com>
To: Lennart Borgman <lennart.borgman <at> gmail.com>
Cc: 5345 <at> debbugs.gnu.org, Reiner Steib <Reiner.Steib <at> gmx.de>
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight 
	checkout
Date: Sun, 10 Jan 2010 21:21:47 +0100

On Sun, Jan 10, 2010 at 21:10, Lennart Borgman
<lennart.borgman <at> gmail.com> wrote:

> Hm. Are you sure?

No. Are you sure that he did?

> I think the RFC assumes that the leading "/" is not part of the
> directory name, so file://<host>/<path> with <host> = localhost and
> <path> = /etc/fstab (etc/fstab relative to /)

I certainly read that as written: the leading / is not part of the filename.

>> He said that he thinks that
>> the RFC assumes that the path does not start with a slash.
>
> Yes.

Then, what are we discussing?

> Please explain. How do you get file:/// on w32?

Using the same interpretation (though I'm not sure I agree with it):
the absolute filename does not start with a slash. Surely you don't
want to remove C from C:/my/path?

BTW, please take a look at
http://en.wikipedia.org/wiki/File_URI_scheme  (even if you don't trust
the Wikipedia, the entry is quite informative).

    Juanma

Message #88 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Lennart Borgman <lennart.borgman <at> gmail.com>
To: Juanma Barranquero <lekktu <at> gmail.com>
Cc: 5345 <at> debbugs.gnu.org, Reiner Steib <Reiner.Steib <at> gmx.de>
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight 
	checkout
Date: Sun, 10 Jan 2010 21:35:41 +0100

On Sun, Jan 10, 2010 at 9:21 PM, Juanma Barranquero <lekktu <at> gmail.com> wrote:
> On Sun, Jan 10, 2010 at 21:10, Lennart Borgman
> <lennart.borgman <at> gmail.com> wrote:
>
>> Hm. Are you sure?
>
> No. Are you sure that he did?

Ehum. Yes. But it does not matter.

>> I think the RFC assumes that the leading "/" is not part of the
>> directory name, so file://<host>/<path> with <host> = localhost and
>> <path> = /etc/fstab (etc/fstab relative to /)
>
> I certainly read that as written: the leading / is not part of the filename.
>
>>> He said that he thinks that
>>> the RFC assumes that the path does not start with a slash.
>>
>> Yes.
>
> Then, what are we discussing?
>
>> Please explain. How do you get file:/// on w32?
>
> Using the same interpretation (though I'm not sure I agree with it):
> the absolute filename does not start with a slash. Surely you don't
> want to remove C from C:/my/path?
>
> BTW, please take a look at
> http://en.wikipedia.org/wiki/File_URI_scheme  (even if you don't trust
> the Wikipedia, the entry is quite informative).

Ah, I see what you mean. I still suspect Reiners interpretation is the
same as the one the confused specs, but it does not work on w32 (as
you also said).

Message #91 received at 5345 <at> debbugs.gnu.org (full text, mbox):

From: Reiner Steib <reinersteib+gmane <at> imap.cc>
To: Lennart Borgman <lennart.borgman <at> gmail.com>
Cc: 5345 <at> debbugs.gnu.org
Subject: Re: bug#5345: Password asked when visiting a file in a lightweight
	checkout
Date: Tue, 12 Jan 2010 19:09:16 +0100

On Sun, Jan 10 2010, Lennart Borgman wrote:

> On Sun, Jan 10, 2010 at 9:21 PM, Juanma Barranquero <lekktu <at> gmail.com> wrote:
>> BTW, please take a look at
>> http://en.wikipedia.org/wiki/File_URI_scheme  (even if you don't trust
>> the Wikipedia, the entry is quite informative).
>
> Ah, I see what you mean. I still suspect Reiners interpretation is the
> same as the one the confused specs, but it does not work on w32 (as
> you also said).

Well the relative path (starting from the "top level") for /etc/fstab
is etc/fstab, whereas the one for c:/autoexec.bat is c:/autoexec.bat
(not autoexec.bat nor :/autoexec.bat nor /autoexec.bat).

Omitting the drive make no sense on windows.  Omitting the "/"
separator (<host>/<path>) also makes no sense, because it would be
ambiguous ...

E.g. host = sti, c:/k
-->  ftp://sti/c:/k.txt
-->  ftp://stic:/k.txt --> huch?  host = stic, port = emtpy, path = k.txt?

BTW: RFC 1738 is superseded by RFC 2396 and the latter by RFC 3986
(STD 66).  (I didn't read them, though :-))

Bye, Reiner.
-- 
       ,,,
      (o o)
---ooO-(_)-Ooo---  |  PGP key available  |  http://rsteib.home.pages.de/

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.