Skip to content

Conversation

sviridov
Copy link
Contributor

@sviridov sviridov commented Sep 8, 2014

Same as #112 but without tests refactoring and with interactive run suuport

@sviridov
Copy link
Contributor Author

sviridov commented Sep 8, 2014

@mattdeboard I will update CONTRIBUTING.md later

@mattdeboard
Copy link
Contributor

Perfect. How did you sort out the when window-system bit?

mattdeboard added a commit that referenced this pull request Sep 8, 2014
@mattdeboard mattdeboard merged commit a23bec1 into elixir-editors:master Sep 8, 2014
@sviridov
Copy link
Contributor Author

sviridov commented Sep 8, 2014

@mattdeboard Hm.. It looks like we have one bug here. When I run:

$ EMACS=`evm bin emacs-24.3-bin` cask exec ert-runner --win

It's actually running not under window-system but in terminal.

@mattdeboard
Copy link
Contributor

I ran this command:

matt@matt-x230:/opt/emacs-elixir$ cask --debug exec ert-runner --win 
Debugger entered--Lisp error: (error "Option `--win` not available")
  signal(error ("Option `--win` not available"))
  error("Option `%s` not available" "--win")
#snip

This led me to look at commander, which only recently added support for the ignore parameter for the option function. So perhaps if it's falling back to 24.3 it doesn't have the right version of commander?

@mattdeboard
Copy link
Contributor

This is assuming you're getting the same error I'm getting when I try to run --win under 24.3:

matt@matt-x230:/opt/emacs-elixir$ cask exec ert-runner --win
cask exec: error: [Errno 2] No such file or directory

@mattdeboard
Copy link
Contributor

Right, so I did the following:

evm use emacs-24.3-bin
# Installs all the deps etc. for 24.3:
cask
# Runs ok:
cask exec ert-runner 
# Opens up the GUI but again closes it down immediately:
cask exec ert-runner --win

@sviridov
Copy link
Contributor Author

sviridov commented Sep 8, 2014

Opens up the GUI but again closes it down immediately:

@mattdeboard See #114.

I run tests with:

$ EMACS=`evm bin emacs-24.3-bin` cask exec ert-runner --win

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

Successfully merging this pull request may close these issues.

2 participants