unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Ihor Radchenko <yantar92@gmail.com>
To: m43cap@yandex.com
Cc: emacs-orgmode@gnu.org,  emacs-devel@gnu.org
Subject: Re: wrong type argument with latest org and latest emacs
Date: Tue, 05 Jul 2022 21:04:09 +0800	[thread overview]
Message-ID: <87czejn22e.fsf@localhost> (raw)
In-Reply-To: <87sfnfhm6v.fsf@yandex.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


  reply	other threads:[~2022-07-05 13:04 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-07-05 10:46 wrong type argument with latest org and latest emacs Colin Baxter
2022-07-05 13:04 ` Ihor Radchenko [this message]
2022-07-05 17:51   ` Colin Baxter
2022-07-05 19:24     ` Colin Baxter
2022-07-07  9:20       ` wrong type argument with latest org and [not] " Alain.Cochard
2022-07-07 12:22         ` Colin Baxter
2022-07-07 12:41           ` Alain.Cochard
2022-07-07 17:07             ` Colin Baxter
2022-07-07  9:30     ` wrong type argument with latest org and " Ihor Radchenko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87czejn22e.fsf@localhost \
    --to=yantar92@gmail.com \
    --cc=emacs-devel@gnu.org \
    --cc=emacs-orgmode@gnu.org \
    --cc=m43cap@yandex.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).