monitorjbl / excel-streaming-reader

An easy-to-use implementation of a streaming Excel reader using Apache POI
Apache License 2.0
953 stars 344 forks source link

Any Reason why xml-apis/xerces2 is included? #138

Open schmitch opened 6 years ago

schmitch commented 6 years ago

Hello, is there any reason why the following dependencies are included:

 <dependency>
      <groupId>com.rackspace.apache</groupId>
      <artifactId>xerces2-xsd11</artifactId>
      <version>2.11.1</version>
    </dependency>
    <dependency>
      <groupId>xml-apis</groupId>
      <artifactId>xml-apis</artifactId>
      <version>1.4.01</version>
    </dependency>

?

The project supports only Java 1.7+ so JAXP is there and the project works successfully without them.

schmitch commented 6 years ago

What's even more strange that it looks like apache-poi comes witht the StAX Apis (which this project also uses, but since Java 1.6 they are not needed anymore)