search

doublep / logview

Emacs mode for viewing log files
GNU General Public License v3.0
153 stars 18 forks source link

Auto-Revert mode over Tramp freezes Emacs #38

Closed xmacex closed 2 years ago

xmacex commented 3 years ago

Enabling the auto-revert-mode (o r by default) in a buffer which is looking at a log file over Tramp makes Emacs (almost) unresponsive.

doublep commented 3 years ago

I cannot reproduce this. Can it be related to how the Auto-revert mode is configured? Can you reproduce with emacs -q? Does it perform better if you switch the buffer to Fundamental mode?

rileyrg commented 3 years ago

I can confirm with debian testing and emacs 27.1 a tramp opened file doesn't revert at all eg /var/log/syslog. I chmodded it to 755 and opened without tramp and auto revert works fine. So tramp issue probably but I haven't looked into it yet.

doublep commented 3 years ago

That it doesn't revert is not necessarily a bug, see issue #30. Check variable auto-revert-remote-files.

The question is, does it make Emacs (almost) unresponsive for you?

rileyrg commented 3 years ago

I realise. I was just adding that "out of the box" it doesn't work with local Tramp rather than raise another issue ;) No, it didn't freeze emacs. Nice package.

On Sun, 6 Dec 2020 at 14:39, Paul Pogonyshev notifications@github.com wrote:

That it doesn't revert is not necessarily a bug, see issue #30 https://github.com/doublep/logview/issues/30. Check variable auto-revert-remote-files.

The question is, does it make Emacs (almost) unresponsive for you?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/doublep/logview/issues/38#issuecomment-739504081, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACYTC2RPDOD5ZVNW4NJTV3STOCQVANCNFSM4QUQ36OQ .

rileyrg commented 3 years ago

Per your Q, I just tested with auto-revert-remote-files (it's local but maybe tramp is "remote") with syslog again read-protected and opened with tramp - it doesn't auto revert.

Edit: I added syslog to auto-mode-alist to trigger logview. revert now works - so either I screwed up earlier, likely, or auto-mode-alist switching on made a difference, unlikely;).

doublep commented 3 years ago

Generally, when something doesn't work over Tramp, check if it is different without Logview (e.g. in Fundamental mode). Logview currently contains nothing Tramp-specific at all, so it is quite unlikely that something additional breaks compared to basic (non-Logview) functionality.