-
Notifications
You must be signed in to change notification settings - Fork 859
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
Maven dependency and build management #57
Conversation
👍 |
I like the idea of adding dependency management, but why not go for something a little more concise like gradle that uses maven under the hood? |
@sghill I don't have any issues as far as you provide an actually working patch |
(I'm not actually the maintainer of this repository, anyway. I was adding 👍 because I reviewed it) |
Right, but you're a member of the Rhino community and that counts for On Thu, Mar 21, 2013 at 8:50 AM, Alberto Pose notifications@gh.neting.ccwrote:
|
Yes, I agree :) |
The current version of Rhino in master now supports building using Gradle. That has worked better than Maven because we are better able to support our legacy stuff. |
I've created some Maven configuration files to build Rhino through Maven (in addition to Ant). The changes are meant to give users a drop-in replacement for Ant without requiring to apply any other Maven specific changes, such as, for example, switching to the Maven standard directory layout.