thierryvolpiatto / psession

Yet another package for emacs persistent sessions
67 stars 13 forks source link

[bug] ###autoloads ... #7

Closed rprimus closed 6 years ago

rprimus commented 6 years ago

Mon Apr 9 18:25:35 BST 2018

HI,

Installing psession via Melpa:

line 258 of psession.el incorrectly has an extra s after ###autoload. This is causing the following in psession-autoaloads.el:

(autoload 'psession-delete-winconf "psession" "\
Delete window config CONF from `psession--winconf-alist'.

\(fn CONF)" t nil)
s

(defvar psession-mode nil "\

which produces the following error upon starting emacs:

Debugger entered--Lisp error: (void-variable s)
  eval-buffer(#<buffer  *load*> nil "/Users/rprimus/.emacs.d/elpa/psession-20180408.2125/psession-autoloads.el" nil t)  ; Reading at buffer position 759
  load-with-code-conversion("/Users/rprimus/.emacs.d/elpa/psession-20180408.2125/psession-autoloads.el" "/Users/rprimus/.emacs.d/elpa/psession-20180408.2125/psession-autoloads.el" nil t)
  load("/Users/rprimus/.emacs.d/elpa/psession-20180408.2125/psession-autoloads" nil t)
  package--activate-autoloads-and-load-path([cl-struct-package-desc psession (20180408 2125) "Persistent save of elisp objects." ((emacs (24)) (cl-lib (0 5)) (async (1 9 2))) nil nil "/Users/rprimus/.e$
  package--load-files-for-activation([cl-struct-package-desc psession (20180408 2125) "Persistent save of elisp objects." ((emacs (24)) (cl-lib (0 5)) (async (1 9 2))) nil nil "/Users/rprimus/.emacs.d/$
  package-activate-1([cl-struct-package-desc psession (20180408 2125) "Persistent save of elisp objects." ((emacs (24)) (cl-lib (0 5)) (async (1 9 2))) nil nil "/Users/rprimus/.emacs.d/elpa/psession-20$
  package-activate(psession)
  package-initialize()
  byte-code("\302 \210^H\204\f^@\303\304        \"\210\302\207" [noninteractive load-file-name package-initialize message "Loading %s..."] 3)
  load("/Users/rprimus/.emacs.d/init" t t)
  #[0 "^H\205\266^@     \306=\203^Q^@\307^H\310Q\202?^@ \311=\204^^^@\307^H\312Q\202?^@\313\307\314\315#\203*^@\316\202?^@\313\307\314\317#\203>^@\320\321\322!D\nB^R\323\202?^@\316\324^S\325^A\324\211#$
  command-line()
  normal-top-level()
thierryvolpiatto commented 6 years ago

R Primus notifications@github.com writes:

Mon Apr 9 18:25:35 BST 2018

HI,

line 258 incorrectly uses autoloads.

Huh! Indeed, wonder how I could miss this, many thanks to catch this!

-- Thierry