Hi Michael,

The patch seems to be working, the only thing I noticed is having
http://www.emacswiki.org/emacs/setup-cygwin.el loaded together with the
patched ob-core.el makes tramp prepend "/cygwin" to /tmp/. When I disabled
(require 'setup-cygwin) the patch works as expected. Outside of org/babel
setup-cygwin has no effect on Tramp.

Thanks,
Alex


On Tue, Sep 24, 2013 at 1:40 PM, Michael Albinus <michael.albinus@gmx.de>wrote:

> Alexander Vorobiev <alexander.vorobiev@gmail.com> writes:
>
> > Hi Michael,
>
> Hi Alex,
>
> > Here it is.
>
> Thanks. I believe, the following patch shall cure it:
>
> --8<---------------cut here---------------start------------->8---
> --- a/lisp/ob-core.el
> +++ b/lisp/ob-core.el
> @@ -2735,8 +2735,12 @@
>  value of `temporary-file-directory' temporarily set to the value
>  of `org-babel-temporary-directory'."
>    (if (file-remote-p default-directory)
>        (let ((prefix
> +            ;; We cannot use `temporary-file-directory' as local part
> +            ;; on the remote host, because it might be another OS
> +            ;; there.  So we assume "/tmp", which ought to exist on
> +            ;; relevant architectures.
>               (concat (file-remote-p default-directory)
> -                     (expand-file-name prefix temporary-file-directory))))
> +                     (expand-file-name prefix "/tmp/"))))
>          (make-temp-file prefix nil suffix))
>      (let ((temporary-file-directory
>            (or (and (boundp 'org-babel-temporary-directory)
> --8<---------------cut here---------------end--------------->8---
>
> Could you, please, test?
>
> > Thanks,
> > Alex
>
> Best regards, Michael.
>