-
Notifications
You must be signed in to change notification settings - Fork 169
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
Too many dependencies? #50
Comments
@hrstoyanov are you concerned with the size of the dependencies or keeping track of them? Are you looking for a single fat jar instead using maven to manage them? Thanks. |
Yeah, take a look at jsoup or aftermarket for inspiration. /Hristo Stoyanov
|
It also conflicts with guava 18.0 and above. emptyIterator() in Guava has become a private method, which causes NoMethodFoundException when rendering a template with Map or List structure. There will be no such a problem if you shade the dependency when making a fat jar |
It's older issue but guava dependency is really terrible. Why not to make the library less requiring (and breaking existing dependencies in project where is used). Shadow is good option. |
Here are the dependency 2.1.7 inflicts on your project:
+--- com.hubspot.jinjava:jinjava:2.1.7
| +--- org.slf4j:slf4j-api:1.7.12
| +--- org.javassist:javassist:3.18.2-GA
| +--- org.jsoup:jsoup:1.8.1
| +--- de.odysseus.juel:juel-api:2.2.7
| +--- de.odysseus.juel:juel-impl:2.2.7
| +--- de.odysseus.juel:juel-spi:2.2.7
| +--- org.apache.commons:commons-lang3:3.4
| --- com.google.code.findbugs:annotations:3.0.0
That is a bit too much! Especially problematic are the findbugs "shutup" annotation, but are these jars all that necessary? The most I can put up with is Guava dependence, if at all... Can you try to make it a "single jar" beauty like the freemarker or jsoup projects?
... Staying away from jinjava for now, given no compelling benefits over freemarker for email templates (Unless you can point some?) and all the extra baggage.
PS: I use ansible a lot and I am well aware of jinja2's power.
The text was updated successfully, but these errors were encountered: