-
Notifications
You must be signed in to change notification settings - Fork 30
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
updates to javacomplete #1
base: master
Are you sure you want to change the base?
Conversation
Ant is required in $PATH as it'll be used with supplied cutsom build.xml to list out classpaths
- win32 java expects win32 paths, so javacomplete would fail if java is compiled for windows - this fix will convert the unix paths to windows paths when executing jre/jdk commands
Conflicts: autoload/javacomplete.vim
- missing a: before arg variable - incorrect usage of System in void context
- new file is README.md
I want to merge your commits in https://github.com/Shougo/javacomplete. |
I haven't worked on this for sometime and don't have it setup in my environment so I don't feel comfortable sending a pull request until I can be sure it will apply cleanly onto your codebase. You are free to add my project as a remote and then merge my changes into your local copy. https://help.github.com/categories/18/articles Basically: add my repo as a remotegit remote add james-m-henderson https://github.com/james-m-henderson/javacomplete.git merge all the stuff my repo has that your's doesn'tgit pull james-m-henderson master You will have to handle some merge conflicts. Let me know if you have any problems. James |
OK. I will try it. Thansk. |
I merged your repository. |
I made some changes to help javacomplete work under cygwin.
I also converted the README to markdown.
I also merged in some changes from itszero and tlvince.