Skip to content
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

Remove unused dependencies #55

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Celebrate-future
Copy link

@gezapeti Hi, I am a user of project org.apache.oozie:oozie-client:5.3.0-SNAPSHOT. I found that its pom file introduced 84 dependencies. However, among them, 15 libraries (17%) have not been used by your project (the redundant dependencies are listed below). This PR helps org.apache.oozie:oozie-client:5.3.0-SNAPSHOT lose weight :) I have tested the revised configuration in my local environment. It is safe to remove the unused libraries.

Best regards

Redundant dependencies----


com.google.code.findbugs:jsr305:jar:1.3.9:compile
org.apache.geronimo.specs:geronimo-jms_1.1_spec:jar:1.1.1:test
org.apache.geronimo.specs:geronimo-j2ee-management_1.1_spec:jar:1.0.1:test
org.apache.directory.server:apacheds-i18n:jar:2.0.0-M15:compile
xml-apis:xml-apis:jar:1.4.01:compile
org.apache.xmlgraphics:batik-rasterizer:jar:1.10:compile
org.apache.xmlgraphics:batik-svgrasterizer:jar:1.10:compile
org.apache.xmlgraphics:batik-ext:jar:1.10:compile
org.apache.xmlgraphics:batik-constants:jar:1.10:compile
net.arnx:nashorn-promise:jar:0.1.1:compile
org.slf4j:jul-to-slf4j:jar:1.7.25:compile
org.apache.hadoop:hadoop-annotations:jar:2.6.0:compile
jdk.tools:jdk.tools:jar:1.6:system
javax.xml.bind:jaxb-api:jar:2.3.0:compile
net.jcip:jcip-annotations:jar:1.0:provided

@asalamon74
Copy link
Contributor

Thanks for the patch @Celebrate-future . Unfortunately we don't support pull requests yet. Can you please open a jira and upload the patch there. Thanks.

Can you also please also avoid changing the indentation of the XML file? It's hard to see the real change with he current version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants