unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#16558: 24.3.50; w32-shell-execute does not open url behind proxy
@ 2014-01-26 10:58 Stefan-W. Hahn
  2014-01-26 19:46 ` Eli Zaretskii
  0 siblings, 1 reply; 9+ messages in thread
From: Stefan-W. Hahn @ 2014-01-26 10:58 UTC (permalink / raw)
  To: 16558


In GNU Emacs 24.3.50.1 (i686-pc-mingw32)
 of 2014-01-19 on LEG570
Repository revision: 116068 eggert@cs.ucla.edu-20140119085053-vie4mrd7nf78r4y9
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --enable-checking 'CFLAGS=-O0 -g3' CPPFLAGS=-DGLYPH_DEBUG=1'

Important settings:
  value of $LANG: DEU
  locale-coding-system: cp1252


Because of commit 0c082e1484a3 (git://git.sv.gnu.org/emacs.git), which fixes
bug #16252, I ran into a problem when calling w32-shell-execute or
browse-url on a url (http/https).

Because I have to work behind a NTLM proxy I have to authenticate and
therfor I use Internet Explorer.

Because of the fix in the above commit w32-shell-execute (and also
browse-url) now call Ffile_exists_p on the url. This is translated into
url-file-exists-p, which tries to load the url; this does not work because
of the proxy. (And if this works, when not having a proxy, the file ist
loaded, and then the Interne Explorer is called on the same url; double
loading).

To solve this for me I set:

  (put 'file-exists-p 'url-file-handlers 'ignore)

I'm not sure if this is the right solution in all cases.

With kind regards,
Stefan

-- 
Stefan-W. Hahn                          It is easy to make things.
                                        It is hard to make things simple.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#16558: 24.3.50; w32-shell-execute does not open url behind proxy
  2014-01-26 10:58 bug#16558: 24.3.50; w32-shell-execute does not open url behind proxy Stefan-W. Hahn
@ 2014-01-26 19:46 ` Eli Zaretskii
  2014-01-27 17:07   ` Stefan-W. Hahn
  0 siblings, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2014-01-26 19:46 UTC (permalink / raw)
  To: Stefan-W. Hahn; +Cc: 16558

> Date: Sun, 26 Jan 2014 11:58:19 +0100
> From: "Stefan-W. Hahn" <info@s-hahn.de>
> 
> Because of commit 0c082e1484a3 (git://git.sv.gnu.org/emacs.git), which fixes
> bug #16252, I ran into a problem when calling w32-shell-execute or
> browse-url on a url (http/https).
> 
> Because I have to work behind a NTLM proxy I have to authenticate and
> therfor I use Internet Explorer.
> 
> Because of the fix in the above commit w32-shell-execute (and also
> browse-url) now call Ffile_exists_p on the url. This is translated into
> url-file-exists-p, which tries to load the url; this does not work because
> of the proxy. (And if this works, when not having a proxy, the file ist
> loaded, and then the Interne Explorer is called on the same url; double
> loading).
> 
> To solve this for me I set:
> 
>   (put 'file-exists-p 'url-file-handlers 'ignore)
> 
> I'm not sure if this is the right solution in all cases.

Would it help in your case to disable file handlers when calling
Ffile_exists_p?





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#16558: 24.3.50; w32-shell-execute does not open url behind proxy
  2014-01-26 19:46 ` Eli Zaretskii
@ 2014-01-27 17:07   ` Stefan-W. Hahn
  2014-01-27 17:11     ` Eli Zaretskii
  0 siblings, 1 reply; 9+ messages in thread
From: Stefan-W. Hahn @ 2014-01-27 17:07 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 16558

Mail von Eli Zaretskii, Sun, 26 Jan 2014 at 21:46:38 +0200:

Hello,

> > To solve this for me I set:
> > 
> >   (put 'file-exists-p 'url-file-handlers 'ignore)
> > 
> > I'm not sure if this is the right solution in all cases.
> 
> Would it help in your case to disable file handlers when calling
> Ffile_exists_p?

I'm not sure what you mean. I tried:

(defun my-browse-url (url)
 (let ((file-name-handler-alist nil))
   (w32-shell-execute nil url)))

this works for my case where I can change the code around the calls to
w32-shell-execute or browse-url.

But what to do with calls to this in other code?

Or do you mean to change file-name-handler-alist and to remove
the "https?":

 ("\\`\\(https?\\|ftp\\|file\\|nfs\\)://" . url-file-handler)

With kind regards,
Stefan


-- 
Stefan-W. Hahn                          It is easy to make things.
                                        It is hard to make things simple.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#16558: 24.3.50; w32-shell-execute does not open url behind proxy
  2014-01-27 17:07   ` Stefan-W. Hahn
@ 2014-01-27 17:11     ` Eli Zaretskii
  2014-01-27 17:31       ` Stefan-W. Hahn
  0 siblings, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2014-01-27 17:11 UTC (permalink / raw)
  To: Stefan-W. Hahn; +Cc: 16558

> Date: Mon, 27 Jan 2014 18:07:37 +0100
> From: "Stefan-W. Hahn" <stefan.hahn@s-hahn.de>
> Cc: 16558@debbugs.gnu.org
> 
> > > To solve this for me I set:
> > > 
> > >   (put 'file-exists-p 'url-file-handlers 'ignore)
> > > 
> > > I'm not sure if this is the right solution in all cases.
> > 
> > Would it help in your case to disable file handlers when calling
> > Ffile_exists_p?
> 
> I'm not sure what you mean. I tried:
> 
> (defun my-browse-url (url)
>  (let ((file-name-handler-alist nil))
>    (w32-shell-execute nil url)))
> 
> this works for my case where I can change the code around the calls to
> w32-shell-execute or browse-url.
> 
> But what to do with calls to this in other code?
> 
> Or do you mean to change file-name-handler-alist and to remove
> the "https?":
> 
>  ("\\`\\(https?\\|ftp\\|file\\|nfs\\)://" . url-file-handler)

I meant to disable file handlers inside w32-shell-execute, on the C
level, so that the call to Ffile_exists_p fails for URLs.  Would that
make your use case work again?





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#16558: 24.3.50; w32-shell-execute does not open url behind proxy
  2014-01-27 17:11     ` Eli Zaretskii
@ 2014-01-27 17:31       ` Stefan-W. Hahn
  2014-01-27 17:53         ` Eli Zaretskii
  0 siblings, 1 reply; 9+ messages in thread
From: Stefan-W. Hahn @ 2014-01-27 17:31 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 16558

Mail von Eli Zaretskii, Mon, 27 Jan 2014 at 19:11:28 +0200:

Hello,

> I meant to disable file handlers inside w32-shell-execute, on the C
> level, so that the call to Ffile_exists_p fails for URLs.  Would that

Oh, but this is your domain.

> make your use case work again?

I think so, because you added the call to Ffile_exists_p in your change
(commit 0c082e14). The call to Fexpand_file_name is no problem, it was
already there before your change.

Another use case where a call to Ffile_exists_p does not make any sense is
when calling i.e. outlook with a message id (the url looks like:
   Outlook:00000000EFABE93C5C7E4947B0D4C936C001637FE4782400
) what I do from within org-mode.

Kind regards,
Stefan

-- 
Stefan-W. Hahn                          It is easy to make things.
                                        It is hard to make things simple.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#16558: 24.3.50; w32-shell-execute does not open url behind proxy
  2014-01-27 17:31       ` Stefan-W. Hahn
@ 2014-01-27 17:53         ` Eli Zaretskii
  2014-02-01  9:24           ` Eli Zaretskii
  0 siblings, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2014-01-27 17:53 UTC (permalink / raw)
  To: Stefan-W. Hahn; +Cc: 16558

> Date: Mon, 27 Jan 2014 18:31:04 +0100
> From: "Stefan-W. Hahn" <stefan.hahn@s-hahn.de>
> Cc: 16558@debbugs.gnu.org
> 
> > make your use case work again?
> 
> I think so, because you added the call to Ffile_exists_p in your change
> (commit 0c082e14). The call to Fexpand_file_name is no problem, it was
> already there before your change.

OK, I will make that change soon.

> Another use case where a call to Ffile_exists_p does not make any sense is
> when calling i.e. outlook with a message id (the url looks like:
>    Outlook:00000000EFABE93C5C7E4947B0D4C936C001637FE4782400
> ) what I do from within org-mode.

Alas, there's no way to know if a string names a file, except by
trying to access it as a file.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#16558: 24.3.50; w32-shell-execute does not open url behind proxy
  2014-01-27 17:53         ` Eli Zaretskii
@ 2014-02-01  9:24           ` Eli Zaretskii
  2014-02-09  8:35             ` Stefan-W. Hahn
  0 siblings, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2014-02-01  9:24 UTC (permalink / raw)
  To: stefan.hahn; +Cc: 16558

> Date: Mon, 27 Jan 2014 19:53:46 +0200
> From: Eli Zaretskii <eliz@gnu.org>
> Cc: 16558@debbugs.gnu.org
> 
> > Date: Mon, 27 Jan 2014 18:31:04 +0100
> > From: "Stefan-W. Hahn" <stefan.hahn@s-hahn.de>
> > Cc: 16558@debbugs.gnu.org
> > 
> > > make your use case work again?
> > 
> > I think so, because you added the call to Ffile_exists_p in your change
> > (commit 0c082e14). The call to Fexpand_file_name is no problem, it was
> > already there before your change.
> 
> OK, I will make that change soon.

Done in trunk revision 116230, please see if your problem is now
solved.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#16558: 24.3.50; w32-shell-execute does not open url behind proxy
  2014-02-01  9:24           ` Eli Zaretskii
@ 2014-02-09  8:35             ` Stefan-W. Hahn
  2014-02-09 16:25               ` Eli Zaretskii
  0 siblings, 1 reply; 9+ messages in thread
From: Stefan-W. Hahn @ 2014-02-09  8:35 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 16558

Mail von Eli Zaretskii, Sat, 01 Feb 2014 at 11:24:55 +0200:

Good morning,

> Done in trunk revision 116230, please see if your problem is now
> solved.

I tested this on windows with r116242. It works for my use case.

Thanks.
Stefan

-- 
Stefan-W. Hahn                          It is easy to make things.
                                        It is hard to make things simple.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#16558: 24.3.50; w32-shell-execute does not open url behind proxy
  2014-02-09  8:35             ` Stefan-W. Hahn
@ 2014-02-09 16:25               ` Eli Zaretskii
  0 siblings, 0 replies; 9+ messages in thread
From: Eli Zaretskii @ 2014-02-09 16:25 UTC (permalink / raw)
  To: Stefan-W. Hahn; +Cc: 16558-done

> Date: Sun, 9 Feb 2014 09:35:11 +0100
> From: "Stefan-W. Hahn" <info@s-hahn.de>
> Cc: 16558@debbugs.gnu.org
> 
> Mail von Eli Zaretskii, Sat, 01 Feb 2014 at 11:24:55 +0200:
> 
> Good morning,
> 
> > Done in trunk revision 116230, please see if your problem is now
> > solved.
> 
> I tested this on windows with r116242. It works for my use case.

Thanks, closing.





^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2014-02-09 16:25 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-01-26 10:58 bug#16558: 24.3.50; w32-shell-execute does not open url behind proxy Stefan-W. Hahn
2014-01-26 19:46 ` Eli Zaretskii
2014-01-27 17:07   ` Stefan-W. Hahn
2014-01-27 17:11     ` Eli Zaretskii
2014-01-27 17:31       ` Stefan-W. Hahn
2014-01-27 17:53         ` Eli Zaretskii
2014-02-01  9:24           ` Eli Zaretskii
2014-02-09  8:35             ` Stefan-W. Hahn
2014-02-09 16:25               ` Eli Zaretskii

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).