bug#55357: closed (29.0.50; [PATCH] Fix browse-url-emacs on file:// urls)

[GNU bug Tracking System]

Your message dated Wed, 11 May 2022 11:35:58 -0700
with message-id <A89D0147-7E7D-432B-AE56-85A35F4EBE49@gmail.com>
and subject line Re: bug#55357: 29.0.50; [PATCH] Fix browse-url-emacs on 
file:// urls
has caused the debbugs.gnu.org bug report #55357,
regarding 29.0.50; [PATCH] Fix browse-url-emacs on file:// urls
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
55357: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=55357
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems

--- Begin Message --- Subject: 29.0.50; [PATCH] Fix browse-url-emacs on file:// urls Date: Wed, 11 May 2022 00:05:50 -0700

If you call browse-url-emacs with a file:// url, Emacs does not open the 
correct file:

(browse-url-emacs "file://~/a/b") opens /a/b
(browse-url-emacs "file://a/b") opens /b

Basically unless the path is an absolute path, the first level is chopped 
because url parses it as host and throws it away. This patch concats the host 
back. If the path is absolute, host is empty string so it’s ok.

Yuan

file-url.patch
Description: Binary data

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#55357: 29.0.50; [PATCH] Fix browse-url-emacs on file:// urls
>
>
>
>
> Date: 
>
> Wed, 11 May 2022 11:35:58 -0700
>
>
>
>
>
> > On May 11, 2022, at 11:35 AM, Yuan Fu <casouri@gmail.com> wrote:
> > 
> > 
> > 
> >> On May 11, 2022, at 4:37 AM, Eli Zaretskii <eliz@gnu.org> wrote:
> >> 
> >>> Resent-From: Yuan Fu <casouri@gmail.com>
> >>> Original-Sender: "Debbugs-submit" <debbugs-submit-bounces@debbugs.gnu.org>
> >>> Resent-CC: bug-gnu-emacs@gnu.org
> >>> Resent-Sender: help-debbugs@gnu.org
> >>> From: Yuan Fu <casouri@gmail.com>
> >>> Date: Wed, 11 May 2022 00:05:50 -0700
> >>> 
> >>> 
> >>> [1:text/plain Hide]
> >>> 
> >>> 
> >>> If you call browse-url-emacs with a file:// url, Emacs does not open the 
> >>> correct file:
> >>> 
> >>> (browse-url-emacs "file://~/a/b") opens /a/b
> >>> (browse-url-emacs "file://a/b") opens /b
> >>> 
> >>> Basically unless the path is an absolute path, the first level is chopped 
> >>> because url parses it as host and throws it away. This patch concats the 
> >>> host back. If the path is absolute, host is empty string so it’s ok.
> >> 
> >> I think your URLs lack one or two more slashes. See
> >> 
> >> https://en.wikipedia.org/wiki/File_URI_scheme#How_many_slashes?
> > 
> > Ah! My bad. Closing.
> > 
> > Yuan
> > 
>
>
>
> --- End Message ---