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

Describe variable fails due to missing definition of "helm-bookmark-map" on develop #9549

Closed
smile13241324 opened this issue Sep 6, 2017 · 48 comments

Comments

@smile13241324
Copy link
Collaborator

smile13241324 commented Sep 6, 2017

Description :octocat:

Since the latest update on develop every call of SPC h d v fails due to an undefined variable helm-bookmark-map.

Reproduction guide 🪲

  • Start Emacs
  • Press SPC h d v and select a variable of your choice
  • The error appears

Observed behaviour: 👀 💔
The symbol helm-bookmark-map is referenced but not defined which causes emacs to fail at displaying the variable description. A quick smart search showed me the file is referenced from the helm layer's packages.el only.

Expected behaviour: ❤️ 😄
Show the description

System Info 💻

  • OS: gnu/linux
  • Emacs: 25.2.1
  • Spacemacs: 0.200.9
  • Spacemacs branch: develop (rev. 6d73df7)
  • Graphic display: t
  • Distribution: spacemacs
  • Editing style: vim
  • Completion: helm
  • Layers:
(helm nim scheme
      (auto-completion :variables auto-completion-return-key-behavior 'complete auto-completion-tab-key-behavior 'cycle auto-completion-complete-with-key-sequence nil auto-completion-complete-with-key-sequence-delay 0.0 auto-completion-private-snippets-directory nil auto-completion-enable-snippets-in-popup nil auto-completion-enable-help-tooltip t auto-completion-enable-sort-by-usage t)
      better-defaults
      (clojure :variables clojure-enable-fancify-symbols t)
      gpu graphviz
      (plantuml :variables plantuml-jar-path "~/.plantuml/plantUml.jar" org-plantuml-jar-path "~/.plantuml/plantUml.jar")
      salt ibuffer emacs-lisp pdf-tools cscope gtags
      (org :variables org-want-todo-bindings t org-enable-github-support t org-enable-bootstrap-support t org-enable-reveal-js-support t)
      (shell :variables shell-enable-smart-eshell t shell-default-shell 'ansi-term shell-default-height 60 shell-default-position 'bottom shell-default-full-span nil shell-default-term-shell "/usr/bin/fish")
      syntax-checking version-control semantic themes-megapack
      (c-c++ :variables c-c++-default-mode-for-headers 'c++-mode c-c++-enable-clang-support t c-c++-enable-clang-format-on-save t)
      git github sml pass common-lisp
      (python :variables python-test-runner 'pytest python-enable-yapf-format-on-save t python-sort-imports-on-save t)
      windows-scripts agda jr shell-scripts emoji
      (markdown :variables markdown-live-preview-engine 'vmd)
      bibtex
      (latex :variables latex-enable-auto-fill t latex-enable-magic t latex-enable-folding t)
      lua html
      (javascript :variables javascript-disable-tern-port-files nil)
      react yaml nginx docker csv search-engine restructuredtext selectric octave systemd imenu-list
      (java :variables java-backend 'Meghanada)
      (go :variables go-use-gometalinter t gofmt-command "goimports" go-tab-width 4 go-use-gocheck-for-testing t go-use-test-args "-race -timeout 10s")
      major-modes coq django
      (elm :variables elm-sort-imports-on-save t elm-format-on-save t)
      idris
      (haskell :variables haskell-enable-hindent-style "fundamental" haskell-completion-backend 'dante)
      pandoc sphinx
 parinfer asciidoc ansible cfengine puppet rebox rust xkcd typography vimscript
      (terraform :variables terraform-auto-format-on-save t)
      (ruby :variables ruby-enable-enh-ruby-mode t ruby-version-manager 'rvm)
      ruby-on-rails asm
      (sql :variables sql-capitalize-keywords t)
      perl6 autohotkey
      (elixir :variables flycheck-elixir-credo-strict t)
      faust vagrant erlang protobuf php racket
      (typescript :variables typescript-fmt-tool 'typescript-formatter typescript-fmt-on-save t))
  • System configuration features: XPM JPEG TIFF GIF PNG RSVG IMAGEMAGICK SOUND GPM DBUS GCONF GSETTINGS NOTIFY ACL GNUTLS LIBXML2 FREETYPE M17N_FLT LIBOTF XFT ZLIB TOOLKIT_SCROLL_BARS GTK3 X11 MODULES

Backtrace 🐾

Debugger entered--Lisp error: (void-variable helm-bookmark-map)
  (define-key helm-bookmark-map (kbd "C-d") (quote helm-bookmark-run-delete))
  simpler-helm-bookmark-keybindings()
  run-hooks(helm-mode-hook helm-mode-on-hook)
  helm-mode(1)
  #[0 "\302\303!\210\300\211�\207" [describe-variable this-command helm-mode 1] 2]()
  helm--completing-read-default("Describe variable: " [preview-copy-mml edebug-Trace-fast-mode helm-info-libffi edebug-inside-func helm-source-info-ede helm-info-gnutls sh-test debugger-previous-window semantic-decoration-mode-hook makefile-makepp-font-lock-keywords sfile-p magit-update-vc-modeline at\ point helm-info-automake tramp-compat-temporary-file-directory start-deleted electric-help-undefined edebug-form-begin-marker helm-info-viper edebug-mode-on-hook executable-magicless-file-regexp helm-info-gnutls-guile helm-info-erc edebug-print-trace-before term-dynamic-list-filename-completions edebug-eval-mode-hook helm-show-completion-display-function edebug-mark-
@duianto
Copy link
Collaborator

duianto commented Sep 6, 2017

There's a PR: Fix helm-bookmark initialization #9547

@smile13241324
Copy link
Collaborator Author

Thanks for telling me, didn't see that. Lets hope this gets merged fast.

@bjarkevad
Copy link
Contributor

Pinning helm to melpa stable seems to work on develop:

(defun dotspacemacs/user-init ()
  (push '("melpa-stable" . "stable.melpa.org/packages/") configuration-layer--elpa-archives)
  (push '(helm . "melpa-stable") package-pinned-packages))

@jajpater
Copy link

jajpater commented Sep 7, 2017

Thanks for the tip bjarkevad but it doesn't work for me.

@bjarkevad
Copy link
Contributor

@jajpater: Did you remove .emacs.d/elpa/helm-* and restart spacemacs? This should force a reinstall of helm with the pinned version.

@jajpater
Copy link

jajpater commented Sep 7, 2017

Thanks @bjarkevad That solved it!

@bmag
Copy link
Collaborator

bmag commented Sep 7, 2017

Should be fixed in f221d7e, can you please verify?

@duianto
Copy link
Collaborator

duianto commented Sep 7, 2017

@bmag Confirmed, C-h v and saving SPC f s, works again.

@smile13241324
Copy link
Collaborator Author

@bmag I can also verify that your fix worked, on latest develop I can SPC h d v again.

@jajpater
Copy link

jajpater commented Sep 7, 2017

Yep it works. Issue can be closed.

@bmag
Copy link
Collaborator

bmag commented Sep 7, 2017

If there are any users of master branch who suffer this error: calling SPC f b or adding (require 'helm-bookmark) to user-config should be a proper work-around.

@stradicat
Copy link
Contributor

Workaround confirmed to work.

Stable setup: Emacs 24.5.1, Spacemacs 0.200.9 (master)

@duianto
Copy link
Collaborator

duianto commented Sep 9, 2017

This seems to be a big enough issue, that a hotfix might be needed for the master branch.
Here are some of the actions that doesn't work and only show the message:

simpler-helm-bookmark-keybindings: Symbol’s value as variable is void: helm-bookmark-map

C-h v test RET
C-h f test RET

Save a new buffer:
open a new empty buffer SPC b N
type something: test
save: SPC f s
choose a location and press RET

Switch branch in a magit buffer: bb
choose a branch and press RET

Some gitter comments also mentioned these:

Syncing layers SPC f e R

M-x desktop-save

Copy a file: SPC f c
choose a location and press RET

Rename a file: SPC f R
choose a new name and press RET

M-x magit status
choose a repository, ex .emacs.d and press RET

System Info 💻

  • OS: windows-nt
  • Emacs: 26.0.50
  • Spacemacs: 0.200.9
  • Spacemacs branch: master (rev. 8e1af14)
  • Graphic display: t
  • Distribution: spacemacs
  • Editing style: vim
  • Completion: helm
  • Layers:
(autohotkey auto-completion colors emacs-lisp git helm markdown org version-control)
  • System configuration features: XPM JPEG TIFF GIF PNG RSVG SOUND NOTIFY ACL GNUTLS LIBXML2 ZLIB TOOLKIT_SCROLL_BARS MODULES

@polydevuk
Copy link

Any news on this? Today I innocently ran "Update Packages" and now Spacemacs is unusable for cider-repl loading (C-c C-l) due to “Symbol’s value as variable is void: helm-bookmark-map”. I tried adding the melpa stable fix and reloading helm but I still get the same error. Latest Spacemacs 0.200.9@25.2.1

@duianto
Copy link
Collaborator

duianto commented Sep 9, 2017

@garyheath Currently, the simplest solution is, as stated in the earlier comment by bmag.

Add this command: (require 'helm-bookmark) to the dotspacemacs/user-config() section in your .spacemacs file. Then restart Emacs and the helm-bookmark-map message will go away.

@polydevuk
Copy link

polydevuk commented Sep 9, 2017 via email

@bmag
Copy link
Collaborator

bmag commented Sep 10, 2017

This seems to be a big enough issue, that a hotfix might be needed for the master branch.

Yes, we are aware. syl20bnr is planning to make a hot fix release soon (I don't know exactly when, but hopefully within several days)

@bmag
Copy link
Collaborator

bmag commented Sep 10, 2017

Reopening for visibility. The issue will be closed in the next release.

@liuchong
Copy link
Contributor

Fixed in develop? Will it take long to merge develop into master? 😹

@mtartare
Copy link

I had the same problem when trying to switch branch with magit. Adding (require 'helm-bookmark) worked for me as well.

Bost added a commit to Bost/dotfiles that referenced this issue Oct 17, 2017
@joneshf
Copy link

joneshf commented Oct 21, 2017

Chiming in as another person on master, updated packages after a while, and hit this bug.

I'm curious what the status of fixing master is.

@macalinao
Copy link

Please fix this. It's breaking my workflow.

@tuxayo
Copy link

tuxayo commented Oct 24, 2017

@macalinao The problem is known and fixed in the develop branch, no need to add messages which could be replaced by upvoting the issue or some of latest comments. (because they dilute the messages related to how to fix the issue and release the fix)

Someone correct me if I'm wrong, but the best thing to do to recover the functionality + helping to ship it would be to switch on the develop branch and report eventual bugs (or just enjoy the fix). So the next release will come sooner.
Or funding the project so maintainers could spent more time on it.
Or creating a bounty on this specific issue.

@what-the-functor
Copy link
Contributor

A newbie just posted on Gitter about this bug. The fix is trivial for those whom are used to Spacemacs. However this bug has the potential to make a bad impression on new users.

It has been over a month since it was determined that there should be a hotfix. What's holding up the hot fix? Anything I can do to help?

@stradicat
Copy link
Contributor

stradicat commented Oct 25, 2017 via email

@smile13241324
Copy link
Collaborator Author

As I see that many of you are getting pretty frustrated with the state on master please consider:
• We are all doing this work in our freetime
• ‎We all have jobs
• ‎We all have families

To be honest currently we have more issues then we have hands to fix. Therefore everybody focus on develop rather than on the old master.

Speaking of hands ... you know spacemacs is community driven, that means we expect and need our users to get involved in the project.

So if you like spacemacs please consider
• If your the programming type fetch one of the random issues from the issue tracker like your generic org export issue or some layer not working properly ticket.
• ‎If you are unsure pick one of the various documentation issues from #9476. This is also a pretty good way to get more familiar with your editor.
• ‎If you are unsure of what is to be done for an issue you can still ask one of the team normally we are pretty fast in responding to sincere questions.

Consider the more stuff we contributors fix the more time the colaborators have for making hotfixes and working on the core. And this is what you guys are all waiting for. So please get involved.

@what-the-functor
Copy link
Contributor

Good points @smile13241324.

We're all developers here, and I assume we all have a good understanding of what 'hotfix' means. This bug is labelled as fixed on develop. What this tells me, is that the fix is waiting on someone to merge or cherry-pick the specific parts needed for a release.

As you said this is a community driven project. The community has fixed the bug; and we are waiting on someone to do the actual release. If there is anything holding this back, please list it here so we (collectively) can contribute toward this fix.

@smile13241324
Copy link
Collaborator Author

@syl20bnr Can you tell when you will be able to provide a hotfix for this on master? This issue seems to become pretty urgent, judging from the amount of people commenting here.

Is there anything blocking you where we could help?

@tuxayo
Copy link

tuxayo commented Oct 26, 2017

Has anyone tried to backport this fix to master an check that it doesn't break anything? That's one step that the maintainer.s wouldn't have to do.

@collinalexbell
Copy link

collinalexbell commented Oct 27, 2017

I can prepare the hotfix and do a backport, since I am installing spacemacs on a new machine today. What is the fix's commit# in develop?

@vbauerster
Copy link

I'm a neovim user trying out spacemacs at the moment. The issue affects go layer: most of go-guru commands do not work out of the box. Also customize-variable do not popup preference window.
Hope you fix this soon, because it is very discouraging for newcommers.

@tuxayo
Copy link

tuxayo commented Oct 29, 2017

@SlightlyCyborg

What is the fix's commit# in develop?

It seems to be f221d7e

@syl20bnr
Copy link
Owner

Going to release a 200.10 with the hotfix.

@antonpaisov
Copy link

@syl20bnr do you mean today or in general? :)

@syl20bnr
Copy link
Owner

Planned tomorrow sorry, yesterday I worked on the signing of Spacelpa tarballs.

Hum ^_^

@syl20bnr
Copy link
Owner

syl20bnr commented Nov 2, 2017

Yesterday you said tomorrow so just DO IT!

Fixed in 0.200.10 🎉

@fosskers
Copy link
Contributor

fosskers commented Nov 2, 2017

Yup, seems to fix this issue!

@antonpaisov
Copy link

time to close this issue :)

@syl20bnr
Copy link
Owner

Fixed in both branches.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests