Skip to content

Conversation

@Celebrate-future
Copy link

@chenjianjx Hi, I am a user of project com.github.chenjianjx:wsdl2html:4.2.1. I found that its pom file introduced 39 dependencies. However, among them, 22 libraries (56%) have not been used by your project (the redundant dependencies are listed below). Reduce these useless dependencies can help prevent conflicts between library versions. MeanWhile, it can minimize the total added size to projects. It can also help enable advanced scenarios for users of your package.
This PR helps com.github.chenjianjx:wsdl2html:4.2.1 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.sun.xml.stream.buffer:streambuffer:jar:1.5.6:compile
com.sun.xml.messaging.saaj:saaj-impl:jar:1.5.0:compile
org.apache.ant:ant:jar:1.10.2:compile
javax.xml.bind:jaxb-api:jar:2.3.1:compile
com.sun.xml.fastinfoset:FastInfoset:jar:1.2.15:compile
org.glassfish.jaxb:txw2:jar:2.3.1:compile
org.apache.ant:ant-launcher:jar:1.10.2:compile
com.sun.xml.ws:policy:jar:2.7.5:compile
javax.jws:javax.jws-api:jar:1.1:compile
org.glassfish.jaxb:codemodel:jar:2.3.1:compile
com.sun.xml.bind.external:rngom:jar:2.3.1:compile
javax.xml.ws:jaxws-api:jar:2.3.1:compile
com.sun.istack:istack-commons-tools:jar:3.0.7:compile
org.glassfish.jaxb:jaxb-runtime:jar:2.3.1:compile
com.sun.istack:istack-commons-runtime:jar:3.0.6:compile
javax.annotation:javax.annotation-api:jar:1.3.2:compile
org.glassfish.jaxb:xsom:jar:2.3.1:compile
com.sun.activation:javax.activation:jar:1.2.0:compile
javax.activation:javax.activation-api:jar:1.2.0:compile
com.sun.xml.dtd-parser:dtd-parser:jar:1.4:compile
javax.xml.soap:javax.xml.soap-api:jar:1.4.0:compile
com.sun.xml.bind.external:relaxng-datatype:jar:2.3.1:compile

@chenjianjx
Copy link
Owner

Hi,

  • with what tool did you find they are useless?
  • did you test it in both jdk 8 and jdk 11? JDK 11 changed a lot when it comes to xml

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