-
Notifications
You must be signed in to change notification settings - Fork 22
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 support for local JARs #55
Comments
I don't care for config.jar "relative/path/to/my.jar"
config.jar "file://absolute/path/to/my.jar"
config.jar "http://example.com/unpublished.jar"
config.jar "http://example.com/path/to/project/source/where/pom/resides/" etc. Basically, if it matches Keep in mind we're not trying to validate a potential coordinate, just determine which of the two it might be. |
Here's a shorter version of the coordinate-matching regexp: /((^|\:)\w+[\w\.\-]*){3,}$/ Must be at-least three parts, separated by colons, a part must begin with a letter or number. |
It makes the API cleaner, sure. But code would be more complex this way. Plus, we'd have to handle other aspects of it not being a standard coordinate dependency. A On the flip side, your API idea could enable JAR hosting without using Sonatype Nexus or another type of Maven repo. I think each JAR would require a pom.xml so we can make sure all dependencies are taken care of, right? Is that a reasonable expectation for local JARs or unpublished projects? |
@ckrailo usually if you have a project not publishing artifacts, it also makes it a pain to obtain To me it sounds like the resolution issue outweights the benefits of overloading |
It's actually fairly well factored (IMO). I think it would just be adding an Since If we're talking about purely that I've downloaded |
@michaelklishin mentioned that supporting local JARs can be a lifesaver:
@sam: Seems like we could add a config.local_jar command that takes a relative or absolute path to a JAR and it would handle this use case.
Thoughts?
The text was updated successfully, but these errors were encountered: