[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [O] org mode R remote code evaluation

From: Alexander Vorobiev
Subject: Re: [O] org mode R remote code evaluation
Date: Tue, 24 Sep 2013 23:24:33 -0500

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.


On Tue, Sep 24, 2013 at 1:40 PM, Michael Albinus <address@hidden> wrote:
Alexander Vorobiev <address@hidden> 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.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]