-
Notifications
You must be signed in to change notification settings - Fork 76
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
Add maven / tycho support (n-1) #40
Comments
@tomotaro1065 I started adding option to build as Maven/tycho (see commit above: just pom.xml files) Can you tell what plugins and features are required from chromedevtools? Let's included in Maven build only those required. |
We need following plugins and features. plugins/org.chromium.debug.corehttps://github.com/Nodeclipse/nodeclipse-1/tree/master/chromedevtools/plugins/org.chromium.debug.core features/org.chromium.debughttps://github.com/Nodeclipse/nodeclipse-1/tree/master/chromedevtools/features/org.chromium.debug 2013/8/19 Paul Verest notifications@github.com
blog: http://d.hatena.ne.jp/tomotaro1065/ |
I will add pom.xml for mentioned projects. What about Node.js bundles? For 0.5 release will you keep them inside or extract outside? |
Already answered in #17 |
Please check if the list is correct. Your list was hard to read. (Check this post for better Markdown usage)
I have added all needed pom's for projects mentioned here (see commit above) However current build
Do you have an idea? |
is
I don't have any idea about the error. 2013/8/22 Paul Verest notifications@github.com
blog: http://d.hatena.ne.jp/tomotaro1065/ |
I got some ideas from http://www.eclipse.org/community/eclipse_newsletter/2013/august/article3.php I need to add something like this:
|
Build succeeded |
I had to rename feature's name in chromium to add ".feature" as they have the same ID as plugins. Let's make all feature to have ".feature" suffix. I have not committed yet. Too late. |
After or before analogical task for coffeescript-eclipse Nodeclipse/coffeescript-eclipse#3
(but exactly in the same maner)
add maven / tycho support
The text was updated successfully, but these errors were encountered: