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: João Távora <joaotavora <at> gmail.com>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#34074: closed (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 message dated Thu, 17 Jan 2019 14:58:09 +0000
with message-id <jjbo98f2uwu.fsf <at> gmail.com>
and subject line Re: bug#34074: 27.0.5; flymake-proc delete temporary directory failed
has caused the debbugs.gnu.org bug report #34074,
regarding 27.0.5; flymake-proc delete temporary directory failed
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> 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: 林宝龙 <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 3 (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 4 (text/html, inline)]

[Message part 5 (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

Previous Next

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