unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#19289: 24.4; Bug in python.el when used in Windows OS
@ 2014-12-06 15:25 David Strip
  2014-12-06 19:36 ` Eli Zaretskii
  2014-12-09  1:34 ` Fabián Ezequiel Gallina
  0 siblings, 2 replies; 3+ messages in thread
From: David Strip @ 2014-12-06 15:25 UTC (permalink / raw)
  To: 19289

[-- Attachment #1: Type: text/html, Size: 4427 bytes --]

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

* bug#19289: 24.4; Bug in python.el when used in Windows OS
  2014-12-06 15:25 bug#19289: 24.4; Bug in python.el when used in Windows OS David Strip
@ 2014-12-06 19:36 ` Eli Zaretskii
  2014-12-09  1:34 ` Fabián Ezequiel Gallina
  1 sibling, 0 replies; 3+ messages in thread
From: Eli Zaretskii @ 2014-12-06 19:36 UTC (permalink / raw)
  To: David Strip; +Cc: 19289

> From: David Strip <david@stripfamily.net>
> Date: Sat, 06 Dec 2014 15:25:58 +0000
> 
> In some sense, the root of the problem is locate-file, called by
> executable-find (both defined in files.el). locate-file does not return quoted
> paths. I'm not sure if the best fix is at the locate-files level, since I'm not
> sure what consequences this might have elsewhere, esp on non-Windows OS's.

locate-file is not supposed to quote file names, because it doesn't
know how the file name will be used.

> The
> most localized fix would be in python.el, changing python-shell-parse-command
> to add the required quotes. In the simplest case, this would be just change the
> format statement from
> 
> (format "%s %s"
> 
> to
> 
> (format "\"%s\" %s"

I think python-shell-parse-command should use shell-quote-argument
instead.

Thank you for your report.





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

* bug#19289: 24.4; Bug in python.el when used in Windows OS
  2014-12-06 15:25 bug#19289: 24.4; Bug in python.el when used in Windows OS David Strip
  2014-12-06 19:36 ` Eli Zaretskii
@ 2014-12-09  1:34 ` Fabián Ezequiel Gallina
  1 sibling, 0 replies; 3+ messages in thread
From: Fabián Ezequiel Gallina @ 2014-12-09  1:34 UTC (permalink / raw)
  To: 19289-done


Fixed at c2db939 in the emacs-24 branch.


Thanks,
Fabián





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

end of thread, other threads:[~2014-12-09  1:34 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-12-06 15:25 bug#19289: 24.4; Bug in python.el when used in Windows OS David Strip
2014-12-06 19:36 ` Eli Zaretskii
2014-12-09  1:34 ` Fabián Ezequiel Gallina

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).