1. jython
  2. jython

Pull requests

#23 Declined
Repository
santa4nt
Branch
default
Repository
jython
Branch
default

Update/fix Intellij IDEA project files

Author
  1. Santoso Wijaya
Reviewers
Description

The project files for IDEA have some errors and missing dependencies, as is. I'm creating a pull request for changes that I made on my forked clone to get IDEA to build cleanly (using the built-in ant plugin, not its default Make tool) and run/debug a simple jython -c "print 'Hello, world!'" all from within IDEA.

Also opened as issue 2113.

  • Learn about pull requests

Comments (3)

  1. Sven Tennie

    Hey Santoso,

    Thank you for your effort to make Jython development easier!

    I'm not part of the jython-core team, but I played a bit around with Jython and Intellij last autumn...

    There is a way to import the eclipse project into Intellij: https://wiki.python.org/jython/JythonDeveloperGuide/IntellijNotes

    The benefit of this way is, that you have to configure project dependencies (jar files in project classpath) only once and not twice (for Intellij AND eclipse) when something changes...

    What do you think about it? What are the benefits of your solution?

    Best regards

    Sven

  2. Santoso Wijaya author

    Hi Sven,

    I didn't realize that guide also existed. I'll be fine doing it either way, as long as it's obvious.

    The reason I did it this way was because I noticed, after cloning a clean copy, the IDEA files being there already. So I opened the project directly and noticed all the broken dependencies.

    In that guide, there are even steps that first need to be done to remove the currently existing IDEA project files:

    rm -rf .idea/
    rm Jython27.iml
    

    So if we want to go with the import-eclipse method, we might as well remove all traces of current IDEA project files from the repository.

    edit: Following the guide you linked, it does indeed feel more straightforward and more maintainable in the long run. I'll open a new pull request for removing the existing IDEA artifacts from the repository.

    Regards, Santoso.