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

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: 林宝龙 <lbl52001 <at> gmail.com>
Subject: bug#34074: closed (Re: bug#34074: 27.0.5; flymake-proc delete
 temporary directory failed)
Date: Thu, 17 Jan 2019 14:59:03 +0000

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

Your bug report

#34074: 27.0.5; flymake-proc delete temporary directory failed

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 34074 <at> debbugs.gnu.org.

-- 
34074: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=34074
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

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

[Message part 3 (message/rfc822, inline)]

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 4 (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 5 (text/html, inline)]

Previous Next

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