all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Michael Albinus <michael.albinus@gmx.de>
To: James Ferguson <james@faff.org>
Cc: Lars Ingebrigtsen <larsi@gnus.org>, 57676@debbugs.gnu.org
Subject: bug#57676: 29.0.50; Tramp: Unable to close vterm buffer with remote default-directory
Date: Sat, 10 Sep 2022 13:48:56 +0200	[thread overview]
Message-ID: <87leqrwitz.fsf@gmx.de> (raw)
In-Reply-To: <CAMtGdS=B9b1ooEGFbZ8WtJkyfebgeGkLO+sTZB8mp+GRdnPgiQ@mail.gmail.com> (James Ferguson's message of "Thu, 8 Sep 2022 11:20:10 -0400")

James Ferguson <james@faff.org> writes:

Hi James,

> That's definitely what raises the error, though the comment implies to
> me that it's unexpected/undesirable that it is triggered.

Indeed. Your backtrace from the original report shows

> Debugger entered--Lisp error: (file-error "File ‘.’ must be absolute")
>   signal(file-error ("File ‘.’ must be absolute"))
>   tramp-error((tramp-file-name "sshx" nil nil "4x02" nil "." nil) file-error "File `%s' must be absolute" ".")
>   tramp-file-name-unify((tramp-file-name "sshx" nil nil "4x02" nil "." nil) ".")
>   tramp-get-file-property((tramp-file-name "sshx" nil nil "4x02" nil "." nil) "." "file-truename")
>   tramp-flush-file-properties((tramp-file-name "sshx" nil nil "4x02" nil "." nil) ".")
>   tramp-flush-file-function()
>   kill-buffer(#<buffer Terminal-4x02 volta@4x02 (10.0.154.60) - byobu>)

We see, that `tramp-flush-file-properties' is called with
`default-directory' "." in #<buffer Terminal-4x02 volta@4x02 (10.0.154.60) - byobu>

This is obviously an error, because the docstring of `default-directory'
says

--8<---------------cut here---------------start------------->8---
Name of default directory of current buffer.
It should be an absolute directory name; on GNU and Unix systems,
these names start with ‘/’ or ‘~’ and end with ‘/’.
--8<---------------cut here---------------end--------------->8---

Perhaps you could report this to the maintainer of vterm.

For the time being, I've pushed a fix to Emacs master which expands
`default-directory' inside `tramp-flush-file-properties'. Could you,
pls, check?

Best regards, Michael.





  reply	other threads:[~2022-09-10 11:48 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-08 12:48 bug#57676: 29.0.50; Tramp: Unable to close vterm buffer with remote default-directory James Ferguson
2022-09-08 12:50 ` bug#57676: further info James Ferguson
2022-09-08 13:13 ` bug#57676: 29.0.50; Tramp: Unable to close vterm buffer with remote default-directory Lars Ingebrigtsen
2022-09-08 15:20   ` James Ferguson
2022-09-10 11:48     ` Michael Albinus [this message]
2022-09-10 20:50       ` James Ferguson
2022-09-11 13:14         ` Michael Albinus

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

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

  git send-email \
    --in-reply-to=87leqrwitz.fsf@gmx.de \
    --to=michael.albinus@gmx.de \
    --cc=57676@debbugs.gnu.org \
    --cc=james@faff.org \
    --cc=larsi@gnus.org \
    /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 external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.