Re: wrong type argument with latest org and latest emacs

[Ihor Radchenko <yantar92@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 837 bytes --]

Colin Baxter <m43cap@yandex.com> writes:

> I'm sending this to emacs.orgmode and emacs.devel lists.
>
> With the latest emacs:
>
>
> Debugger entered--Lisp error: (wrong-type-argument stringp (wrong-type-argument stringp nil))
>   format-message((wrong-type-argument stringp nil))
>   apply(format-message (wrong-type-argument stringp nil))
>   error((wrong-type-argument stringp nil))
>   #f(compiled-function (fun) #<bytecode -0x1724f6e>)(org-babel-remove-temporary-stable-directory)
>   run-hook-wrapped(#f(compiled-function (fun) #<bytecode -0x1724f6e>) org-babel-remove-temporary-stable-directory)

Thanks for reporting!
This likely caused by recent commit of mine on systems with no write
access to remote directory (at least, I am unable to reproduce the steps
on my system).

Can you please try the attached patch?

Best,
Ihor


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-ob-core-Fix-nil-value-of-org-babel-temporary-stable-.patch --]
[-- Type: text/x-patch, Size: 1203 bytes --]

From ddf6278e8fcbaa4939539277b111061b7c00f550 Mon Sep 17 00:00:00 2001
Message-Id: <ddf6278e8fcbaa4939539277b111061b7c00f550.1657026153.git.yantar92@gmail.com>
From: Ihor Radchenko <yantar92@gmail.com>
Date: Tue, 5 Jul 2022 21:00:24 +0800
Subject: [PATCH] ob-core: Fix nil value of
 `org-babel-temporary-stable-directory'

* lisp/ob-core.el: Fallback the value of
`org-babel-temporary-stable-directory' to
`org-babel-temporary-directory' if there are issues with directory
creation.

Fixes https://yhetil.org/emacs-devel/87sfnfhm6v.fsf@yandex.com
---
 lisp/ob-core.el | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/lisp/ob-core.el b/lisp/ob-core.el
index 6c379c121..aaf895d74 100644
--- a/lisp/ob-core.el
+++ b/lisp/ob-core.el
@@ -3167,7 +3167,8 @@   (defvar org-babel-temporary-stable-directory
 	     (expand-file-name
               "babel-stable"
               (temporary-file-directory)))
-          (t nil)))
+          ;; Fallback if things do not work.
+          (t org-babel-temporary-directory)))
     "Directory to hold temporary files created to execute code blocks.
 Used by `org-babel-temp-file'.  This directory will be removed on
 Emacs shutdown."))
-- 
2.35.1