abingham / traad

An JSON+HTTP server for the rope Python refactoring library
MIT License
107 stars 20 forks source link

Try without system installation #4

Closed tkf closed 12 years ago

tkf commented 12 years ago

Hi, this is awesome project! I feel limitation on the Pymacs's serial execution nature, too. This is great you are breaking the limitation.

BTW, I want install traad using el-get without installing it system-wide (or using virtualenv) because it is easy to sync elisp and python parts this way. The problem is that there is no easy way to get executable without installing it. I suggest two solutions:

  1. Put traadserver.py on the top level which contains

    #!/usr/bin/env python
    from traad.server import main
    main()
  2. Allow traad-host to take a list. This way, I can do:

    (setq traad-host '("python" "-m" "traad.server"))
abingham commented 12 years ago

Thanks, I hope you find it useful :)

I have to admit that I don't fully understand everything you're asking for, but that's mostly because I know almost nothing about el-get, elpa, and the other emacs packaging solutions. With that said, I'm all in favor of making traad easier to use, and I want to learn about these things.

Your proposal sounds fairly straightforward, so would you mind coding it up and sending me a patch? That'll help me understand it, its relationship to both emacs and python, and that kind of stuff. I'm sure we can ultimately settle on a solution that we're both happy with.

Austin

On Mon, Jul 23, 2012 at 6:55 PM, Takafumi Arakaki reply@reply.github.com wrote:

Hi, this is awesome project! I feel limitation on the Pymacs's serial execution nature, too. This is great you are breaking the limitation.

BTW, I want install traad using el-get without installing it system-wide (or using virtualenv) because it is easy to sync elisp and python parts this way. The problem is that there is no easy way to get executable without installing it. I suggest two solutions:

  1. Put traadserver.py on the top level which contains

    #!/usr/bin/env python
    from traad.server import main
    main()
  2. Allow traad-host to take a list. This way, I can do:

    (setq traad-host '("python" "-m" "traad.server"))

Reply to this email directly or view it on GitHub: https://github.com/abingham/traad/issues/4

tkf commented 12 years ago

I chose the second solution, because you might want to put something like this in the future:

(setq traad-host '("traad" "--some-optional=parameter"))
tkf commented 12 years ago

Oops. I mean traad-server-program, not traad-host.