-
Notifications
You must be signed in to change notification settings - Fork 116
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
2.4.x #69
base: 2.4.x
Are you sure you want to change the base?
2.4.x #69
Conversation
Could you please provide a unit test to demonstrate what is wrong with IVY-1576 fix? |
I suggest closing this PR. The main problem here is that programmatic use should not be adressed by adding CLI options which make no sense. Also, if resolve() ends with errors, it's best to throw an exception encapsulating whatever relevant information is available, rather than blithely carrying on. |
I think it would be great "not" to have an additional CLI option, but then you cannot have a System.exit(1) terminating the client code (https://github.com/apache/ant-ivy/blob/2.4.x/src/java/org/apache/ivy/Main.java#L313) just because of an error. The errors (and the successes) can be queried out of the ResolveReport. A client can programmatically make decisions based on the data returned. And the run() methods still throw their respective exceptions which could be caught by the client consumer. The new option is to alleviate the problematic design of having an error terminate a client consumer with a System.exit. Perhaps a design would be to copy/paste Main.java to IvyClient.java and remove the System.exit - but then it would be less normalized, which has its own flaws. |
I also spam this pull request. (why not open issue tracker here?) Ivy CLI says: -types comma separated list of accepted artifact types java -jar ivy-2.4.0.jar -cache ./ivy -types source,jar ... does NOT work. What does work is a white space separated list of artifacts. |
Issue tracker for the project is JIRA. Apache has own Git infrastructure, GitHub is used to increase exposure and as easy means to suggest code changes (rather than uploading patches). Please check the latest Ivy build. I cannot verify your report right now, but the value of types should be split correctly in master. |
@supertick FYI, Ivy 2.5.0 RC is on the way, @nlalevee has kindly taken the role of release manager. Your latest commits are not merged to master, though. If you are interested in using 2.5.0, please let us know. |
How should we let you know? Here? I am interested in using 2.5.0. |
Check out the nightlies |
No description provided.