search

rejeep / commander.el

Emacs command line parser
GNU General Public License v3.0
56 stars 8 forks source link

Not working in Emacs 24.1 and 24.2 #4

Closed rejeep closed 10 years ago

rejeep commented 10 years ago

Reproduce by running tests with Emacs version.

rejeep commented 10 years ago

@jorgenschaefer, I pushed this fix that might fix it https://github.com/rejeep/commander.el/commit/23d600e7ee6a1dc4939c87cbf7cc258b2d723238

It's really hard to find this bug because it works when not byte compiled, but it also works if I byte compile it myself. It's only when installed via Elpa that it fails.

Let's try this out when a new release of Commander is on Melpa.

rejeep commented 10 years ago

@jorgenschaefer The error seems to have been fixed? But at the same time moved to some add-to-list in Ecukes :)

jorgenschaefer commented 10 years ago

Yep, this error is fixed. Next one is in ecukes-parse-intro in ecukes.

In my original repo, followed by a Wrong type argument: sequencep, 257 error when executing the ecukes command And I go to word "return", same behavior, fixed by byte compiling myself … and another one when pressing a key … should I (re-)open the issue on ecukes?

rejeep commented 10 years ago

should I (re-)open the issue on ecukes?

Do that and I will go through the whole project and update all of those...