rspec-toggle-spec-and-target error about rspec-mode

[singram]

My apologies, I'm fairly inexperienced in the specifics of github.

I tracked the issue down to a missing definition for 'first'. I was unable to find and mention of it in the emacs documentation, .el files or google. Swapping out all instances of 'first' for 'car' (which seemed to be the intention) fixed all the issues for me.

A pull request with the fixes is in another thread.

[pedz]

FYI

I have:

(defalias 'first 'car)

in lisp/emacs-lisp/cl.el for me (emacs 24-2).

On Mar 9, 2013, at 7:30 PM, singram wrote:

My apologies, I'm fairly inexperienced in the specifics of github.

I tracked the issue down to a missing definition for 'first'. I was unable to find and mention of it in the emacs documentation, .el files or google. Swapping out all instances of 'first' for 'car' (which seemed to be the intention) fixed all the issues for me.

A pull request with the fixes is in another thread.

—
Reply to this email directly or view it on GitHub.

[singram]

Thanks Perry for the quick response. I'm running 24.1.1 and appear to have the same alias in that file.

Do you know if this is a package that explicitly needs requiring or something that should auto-load?

[pedz]

I see I have a few packages that require 'cl. rinari is the package that most of the other users of rspec-mode probably load.

I would change your pull request to just require 'cl. That looks like the smallest change.

[singram]

Thanks Perry for the feedback. Have submitted a new pull request along the lines you suggested to resolve the issue.