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

Composer Eclipse Editor #25

Closed
wants to merge 22 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@gossi
Collaborator

gossi commented Mar 9, 2013

Here it is, Composer Editor.

Changes:

  • Separated the ui into it's own plugin com.dubture.composer.ui
  • Deactivated the Composer Menu (it was basically everywhere, and annoying everywhere) functionality superseeded by the Composer Editor anyway

Things to rework:

  • Update the submodule back to pulse00/Composer-Java-Bindings
  • Executing Composer (it's a holy mess at the moment) see #15

Broken things:

  • The Composer-Builder Stuff is broken, because I changed the Composer Model (see pulse00/Composer-Java-Bindings#3)
    • Basically, rework Automatic Build-Path Management (issue will follow)
  • Require Dialog broken, actually can be deleted

fixes #12
fixes #20
fixes #21

gossi

url = git://github.com/pulse00/Composer-Java-Bindings.git
[submodule "org.getcomposer"]
path = org.getcomposer
url = git://github.com/gossi/Composer-Java-Bindings.git

This comment has been minimized.

@pulse00

pulse00 Mar 10, 2013

Owner

@gossi We need to change the submodule path back after the merge. Or even better, we could completely remove the submodule setup , as it's pretty much useless in the project setup. Simply adding the Java-Bindings in the Eclipse workspace works too.

This comment has been minimized.

@gossi

gossi Mar 10, 2013

Collaborator

Yes, I realized this during development after changing this submodule to my fork. It isn't necessary as long as both projects are checked out in eclipse as projects. The run configuration is capable of recognizing that this is a local plugin.

@ghost ghost assigned gossi Mar 23, 2013

@gossi

This comment has been minimized.

Collaborator

gossi commented Apr 1, 2013

Merged into develop branch.

@gossi gossi closed this Apr 1, 2013

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