Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error msg "Key sequence M - m SPC h SPC d SPC m starts with non-prefix key" #2671

Closed
1 task done
sunlin7 opened this issue May 15, 2024 · 5 comments
Closed
1 task done
Labels

Comments

@sunlin7
Copy link
Contributor

sunlin7 commented May 15, 2024

What happened?

There are error msg "Key sequence M - m SPC h SPC d SPC m starts with non-prefix key" for the describe-mode was binded to "M-m h d m" before loading helm.
Here are the lines to reproduce the issue, which will bind the describe-mode before loading the helm:

(global-set-key "M-m h d m" 'describe-mode)
(load-file "<PATH>/helm-core-20240510.1815/helm-core-autoloads.el")
(load-file "<PATH>/helm-20240510.1833/helm-autoloads.el")
(call-interactively 'helm)

The helm has special code to deal the describe-mode and leads the issue:

helm/helm-core.el

Lines 479 to 481 in 6d23a65

;; Use `describe-mode' key in `global-map'.
(dolist (k (where-is-internal #'describe-mode global-map))
(define-key map k #'helm-help))

(defvar helm-map
;;...
    (define-key map (kbd "M-m")        #'helm-toggle-all-marks)
;;...
    ;; Use `describe-mode' key in `global-map'.
    (dolist (k (where-is-internal #'describe-mode global-map))
      (define-key map k #'helm-help))
;;...

This issue acctually happened on Spacemacs, was disscussed in syl20bnr/spacemacs#16397.

How to reproduce?

N/A

Helm Version

Master branch

Emacs Version

Emacs-29.1

OS

GNU/Linux

Relevant backtrace (if possible)

No response

Minimal configuration

  • I agree using a minimal configuration
@sunlin7 sunlin7 added the bug label May 15, 2024
@thierryvolpiatto
Copy link
Member

thierryvolpiatto commented May 15, 2024 via email

@sunlin7
Copy link
Contributor Author

sunlin7 commented May 15, 2024

Hi @thierryvolpiatto
Currently helm will stop to work totally if user binding describe-mode to M-m h d m before loading helm.
The issue only affects one key-binding, helm shoulbe be able to continue works for other features.
So if helm gives a user-error friendly then continue, it should be better.

@thierryvolpiatto
Copy link
Member

thierryvolpiatto commented May 16, 2024 via email

@michael-heerdegen
Copy link
Contributor

michael-heerdegen commented May 16, 2024 via email

@sunlin7
Copy link
Contributor Author

sunlin7 commented May 16, 2024

Hi @thierryvolpiatto @michael-heerdegen
Thanks for your explaination, and close this ticket for the original issue hacked by alias describe-mode to new name.
Refer: syl20bnr/spacemacs#16397

@sunlin7 sunlin7 closed this as completed May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants