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

elfeed-update-feed does not complete #64

Closed
absnc opened this issue Dec 28, 2014 · 3 comments
Closed

elfeed-update-feed does not complete #64

absnc opened this issue Dec 28, 2014 · 3 comments

Comments

@absnc
Copy link

absnc commented Dec 28, 2014

Hi,

I'm facing an issue with updating the majority of my feeds.

  • Each time I try to update one of them with elfeed-update-feed, only one single entry is getting fetched.
    Then I'm getting the following error in my Messages buffer :
Elfeed update failed for http://feeds.mashable.com/Mashable?format=xml: (void-variable feed-title)
  • Re-running the command again lets me fetch the following missing entry, and so on.

Sorry but, wasn't able to find answers using the void-variable / feed-title keywords. Thanks.

@skeeto
Copy link
Owner

skeeto commented Dec 29, 2014

What version of Emacs are you running? From the error message I'm guessing either your version doesn't have lexical scope (i.e. Emacs 23.4 and earlier) or you've copy-pasted tagger code (elfeed-make-tagger) to a separate file that's not using lexical scope (lexical-binding).

@absnc
Copy link
Author

absnc commented Dec 29, 2014

Thank you, I was culprit of the latter. It worked well after I deleted this section of my.el file.
I copied elfeed-make-tagger function, as there was an undefined error while I was calling this snippet.

;; (add-hook 'elfeed-new-entry-hook
;;           (elfeed-make-tagger :feed-url "youtube\\.com"
;;                               :add '(youtube)))
  • My version of Emacs is :
    GNU Emacs 24.3.1 (x86_64-apple-darwin, NS apple-appkit-1038.36) of 2013-03-13 on bob.porkrind.org
  • Using elfeed
    elfeed-20141104.1857

@skeeto
Copy link
Owner

skeeto commented Dec 29, 2014

Hmmm, having that snippet in your config should work fine.
elfeed-make-tagger is defined using cl-defun in a lexically scoped file,
which is what sets up the missing feed-title variable. I wonder if your
cl-lib isn't compiled as it should be, causing some macro to be expanded
at the wrong time. Unfortunately macros don't play well with
lexical-binding, since they are expanded and compiled in the context of
the caller.

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

No branches or pull requests

2 participants