search

purcell / emacs.d

An Emacs configuration bundle with batteries included
BSD 2-Clause "Simplified" License
6.87k stars 2.05k forks source link

emacs Got Assertion failed: (derived-mode-p 'ibuffer-mode) #860

Closed bloodofdracula closed 9 months ago

bloodofdracula commented 9 months ago

Using stock starter kit without any adds or delete init or any modification (including custom.el). Open some files from emacsd-29-1 distribution (downloaded from gnu site) C-x C-b (ibuffer) Use / to make some selection Select by file extension option . and enter Select el type of file Got Assertion failed: (derived-mode-p 'ibuffer-mode) See attached screen pictures...

Same with other options..by size, filename..

After selecting . (/ - preceding) . (selecting by file extension) and BEFORE selecting extension el emacs display a buffer in the list and do not display the buffer list..strange

ibuffer-01 ibuffer-02 ibuffer-03 ibuffer-04 ibuffer-05

Using emacs 29.1 Development version 28fb02492c24 on master branch; build date 2023-12-10.

Linux Zorin. Emacs from snap store

purcell commented 9 months ago

Hmm, interesting: if you hit /. very quickly, so that the which-key window doesn't pop up, then this doesn't happen, so I think it's some sort of interaction between which-key and ibuffer. I'll see if I can figure out what the issue is. Interestingly, the equivalent in dired (*.) doesn't have the same issue.

bloodofdracula commented 9 months ago

I did a test on a stock emacs (same from snap without which-key) and it is working as it is supposed to do...Thanks for your answer and help.

purcell commented 9 months ago

I found and fixed it: some kind of weird interaction with the fullframe package, which I plan to replace anyway. So I've just disabled it for that command.

bloodofdracula commented 9 months ago

lisp/init-ibuffer.el

(require-package 'fullframe) (with-eval-after-load 'ibuffer (fullframe ibuffer ibuffer-quit))

So I only need to suppress those lines..right ? Thanks for your nice work.

purcell commented 9 months ago

Yes, but I've pushed that change to this repo.