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

added python compile command #806

wants to merge 1 commit into from


Copy link

@cpaulik cpaulik commented Mar 11, 2015

This makes using compile mode in python easier by doing the following:

  • automatically changes compile command to
    python (buffer-file-name
    for python buffers
  • automatically starts comint mode in python mode to allow interaction
    with debugger

Fixes #805

- automatically changes compile command to
  ```python (buffer-file-name```
  for python buffers
- automatically starts comint mode in python mode to allow interaction
  with debugger
@syl20bnr syl20bnr force-pushed the develop branch 8 times, most recently from fded81b to ef4d48a Compare March 13, 2015 03:51
Copy link

Excellent ! 👍

I reworked it like this:

  • since I removed the advice the key binding is major specific SPC m c c
  • without a universal argument it does not ask for a command, with an universal argument it asks for a command (SPC u SPC m c c)
  • I added SPC m c C to compile and then switch to the comint buffer in insert state.

Commit: 3098a1a

Thank you!
Cherry-picked in develop, you can safely delete your branch.

@syl20bnr syl20bnr closed this Mar 14, 2015
Copy link

oh you got the 400th PR congratulation ! :-)

@cpaulik cpaulik deleted the python-layer-compile branch March 14, 2015 09:41
Copy link
Contributor Author

cpaulik commented Mar 14, 2015

Thanks for making functions out of it, way cleaner.

I tested it and found a problem when the compile-command is already set to e..g a first python file and you want to execute a second python file. I also moved point to the end of Buffer if we are switching to the comint buffer, so we can directly interact with the debugger. see PR #824

Yay 400th 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.

None yet

2 participants