GNU bug report logs - #34074
27.0.5; flymake-proc delete temporary directory failed

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 34074 in the body.
You can then email your comments to 34074 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: 林宝龙 <lbl52001 <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 27.0.5; flymake-proc delete temporary directory failed
Date: Mon, 14 Jan 2019 10:55:28 +0100

[Message part 1 (text/plain, inline)]

The temporary-file-directory on my host is "/tmp/", current function
trancates the first charactor of the directory name, cause the directory
not exist error.

Below patch removes the folder separator of the temporary-file-directory,
works fine in my environment.

diff --git a/lisp/progmodes/flymake-proc.el b/lisp/progmodes/flymake-proc.el
index 7cdbb266fe..f0b979185e 100644
--- a/lisp/progmodes/flymake-proc.el
+++ b/lisp/progmodes/flymake-proc.el
@@ -886,7 +886,7 @@ can also be executed interactively independently of

 (defun flymake-proc--delete-temp-directory (dir-name)
   "Attempt to delete temp dir created by
`flymake-proc-create-temp-with-folder-structure', do not fail on error."
-  (let* ((temp-dir    temporary-file-directory)
+  (let* ((temp-dir    (directory-file-name temporary-file-directory))
         (suffix      (substring dir-name (1+ (length temp-dir)))))

     (while (> (length suffix) 0)

[Message part 2 (text/html, inline)]

Message #8 received at 34074 <at> debbugs.gnu.org (full text, mbox):

From: João Távora <joaotavora <at> gmail.com>
To: 林宝龙 <lbl52001 <at> gmail.com>
Cc: 34074 <at> debbugs.gnu.org, 34074-done <at> debbugs.gnu.org
Subject: Re: bug#34074: 27.0.5; flymake-proc delete temporary directory failed
Date: Thu, 17 Jan 2019 14:58:09 +0000

林宝龙 <lbl52001 <at> gmail.com> writes:

> The temporary-file-directory on my host is "/tmp/", current function
> trancates the first charactor of the directory name, cause the
> directory not exist error.
>
> Below patch removes the folder separator of the temporary-file-directory, works fine in my environment.

Thanks, I pushed a slightly safer fix in

commit 80cbfb61c5a562d51197d6f3068fa5f4cda432b0
Author: João Távora <joaotavora <at> gmail.com>
Date:   Thu Jan 17 14:38:44 2019 +0000

    Fix flymake-proc--delete-temp-directory if temp dir ends in slash

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.