-
Notifications
You must be signed in to change notification settings - Fork 56
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
Dependency addition to Joshua-Decoder/Joshua/pom.xml #239
Comments
from the Apache Incubator general@ discussion. |
@mjpost Joshua still builds with Ant right? Is there a specific requirement for the Maven pom.xml? |
Yep, it's ant-based. It's in the Ivy dependency file, |
Ack On Friday, February 12, 2016, Matt Post notifications@github.com wrote:
Lewis |
I don't think we need to remove it. Doesn't Maven replace Ivy? We could transition at some point. |
Oh, so you want to move away from Ant + Ivy to Maven? On Fri, Feb 12, 2016 at 1:08 PM, Matt Post notifications@github.com wrote:
Lewis |
I wouldn't say I want to, but am definitely willing if someone wants to make the case for straight Maven over ant + ivy. In general, I prefer stability in build tools and am wary of jumping from ship to ship, but if there's a good case for it, I'd happily hear it. I've had some pressure to do so from another source, so maybe it'd be good to consider. |
What an understatement this was. On Fri, Feb 12, 2016 at 7:03 PM, Matt Post notifications@github.com wrote:
Lewis |
This issue can be closed for the time being. it is not required as Maven is not the primary build system. |
Joshua-Decoder/Joshua committer please add this dependency to pom.xml
thank you/
Martin
The text was updated successfully, but these errors were encountered: