[
https://issues.apache.org/jira/browse/SHINDIG-207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12591431#action_12591431
]
David Primmer commented on SHINDIG-207:
---------------------------------------
Regarding the README, I think the debug instructions are good and workable, but
the existing shindig site has developer info on how to setup eclipse and it has
a pointer to basically the same info here:
http://cwiki.apache.org/WICKET/maven-jetty-plugin.html. So maybe this should
just an update of the shindig java developer info and pointer to it in this
README? I also think the suspend=n should be explained so that people know that
it will start the java server independently of any attached debugger. Some may
want y (as it recommends in the above instructions) and some may want n.
Regarding moving the dependencies, and speaking only for the social-api ones,
I'm not sure why you're moving stuff that is really only for social-api
(abdera) to the root. It shouldn't need to be there and just introduces
dependencies where there aren't any. So, yeah, move things like junit and
easymock to the java root, they are global java packages for shindig, but leave
caja and abdera in their respective pom's. (caja and abdera are only examples.
there are others I'm sure)
davep
> simplify dependencies, document how to get eclipse working
> ----------------------------------------------------------
>
> Key: SHINDIG-207
> URL: https://issues.apache.org/jira/browse/SHINDIG-207
> Project: Shindig
> Issue Type: Bug
> Reporter: Brian Eaton
> Attachments: build-cleanup.patch
>
>
> Got a patch for this.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
