rdallasgray / pallet

A package management tool for Emacs, built on Cask.
279 stars 15 forks source link

Wrong type argument when deleting package #18

Closed ghost closed 10 years ago

ghost commented 10 years ago

I'm on ubuntu 12.04, using emacs built from source this pm, and the latest Cask. If this should be x-posted as a Cask issue, I can do that.

When deleting a package from the "package-list-packages" interface, i get the following error: Wrong type argument: stringp, (lambda (ad--addoit-function package-name version) #("Advice function assembled by advice.el." 0 39 (dynamic-docstring-function ad--make-advised-docstring)) (let (ad-return-value) (setq ad-return-value (with-no-warnings (funcall ad--addoit-function package-name))) (pt/maybe-unpack-on-delete package-name) ad-return-value))

Adding (setq pallet-unpack-on-delete nil) to my init.el so that it looks like (require 'cask "~/.cask/cask.el") (cask-initialize) (require 'pallet) (setq pallet-unpack-on-delete nil) does not change this.

There aren't any other user-accessible variables beginning with "pallet-" and the ones starting with "cask-" seem correct.

Cask file before and after trying to delete "ag": (source "melpa" "http://melpa.milkbox.net/packages/")

(depends-on "[cl-struct-package-desc ag (20131103 1957) A front-end for ag ('the silver searcher'), the C ack replacement. [github] nil single melpa nil nil nil]") (depends-on "pallet")

This has also prevented me from upgrading packages.

rdallasgray commented 10 years ago

Hi Tom,

Pallet doesn't at present work correctly on Emacs trunk, owing to quite large changes in package.el. @dudebout contributed a patch in #12, which hasn't been merged as I didn't feel it was quite ready. I am working to fix the issue properly, but if you need to use Emacs trunk, you can use the #12 patch if you wish.

My advice would be to stick with the official release for the moment, though -- it's not always realistic to expect everything to work in a prerelease.

ghost commented 10 years ago

Robert, That worked, thanks! I'll close the issue.

rdallasgray commented 10 years ago

OK, thanks (and sorry for calling you Tom!).