unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#34074: 27.0.5; flymake-proc delete temporary directory failed
@ 2019-01-14  9:55 林宝龙
  2019-01-17 14:58 ` João Távora
  0 siblings, 1 reply; 2+ messages in thread
From: 林宝龙 @ 2019-01-14  9:55 UTC (permalink / raw)
  To: 34074

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

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)

[-- Attachment #2: Type: text/html, Size: 1226 bytes --]

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* bug#34074: 27.0.5; flymake-proc delete temporary directory failed
  2019-01-14  9:55 bug#34074: 27.0.5; flymake-proc delete temporary directory failed 林宝龙
@ 2019-01-17 14:58 ` João Távora
  0 siblings, 0 replies; 2+ messages in thread
From: João Távora @ 2019-01-17 14:58 UTC (permalink / raw)
  To: 林宝龙; +Cc: 34074, 34074-done

林宝龙 <lbl52001@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@gmail.com>
Date:   Thu Jan 17 14:38:44 2019 +0000

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






^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-01-17 14:58 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-01-14  9:55 bug#34074: 27.0.5; flymake-proc delete temporary directory failed 林宝龙
2019-01-17 14:58 ` João Távora

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).