search

rlalfo / google-http-java-client

Automatically exported from code.google.com/p/google-http-java-client
0 stars 0 forks source link

Move XML support to separate google-http-client-xml jar #237

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
External references, such as a standards document, or specification?

http://javadoc.google-http-java-client.googlecode.com/hg/1.15.0-rc/com/google/ap
i/client/xml/package-summary.html

http://javadoc.google-http-java-client.googlecode.com/hg/1.15.0-rc/com/google/ap
i/client/http/xml/package-summary.html

http://javadoc.google-http-java-client.googlecode.com/hg/1.15.0-rc/com/google/ap
i/client/xml/atom/package-summary.html

http://javadoc.google-http-java-client.googlecode.com/hg/1.15.0-rc/com/google/ap
i/client/http/xml/atom/package-summary.html

Java environments (e.g. Java 6, Android 2.3, App Engine, or All)?

All

Please describe the feature requested.

No Java code changes, just a jar packaging change to move all of the XML 
related code to a separate google-http-client-xml jar.  In general, it has 
shown to be a good practice to control the use of third-party dependent 
libraries into separate jars, e.g. what we've done for protobuf support.  
Developers not using the XML support would appreciate not having the xpp3 
library on their classpath.

There is actually a problem for Android developers using Maven because we are 
forced to add xpp3 library as a dependency, but the Android SDK also has those 
classes from the xpp3 library that we are using, so the xpp3 library must be 
explicitly excluded.  It would be nice to limit this pain point to only 
developers that need the XML support.

Note: that means that any library or application that uses the XML support from 
google-http-client will now need to add a new dependency on 
google-http-client-xml.

Original issue reported on code.google.com by yan...@google.com on 24 Jul 2013 at 7:07

GoogleCodeExporter commented 9 years ago

Original comment by yan...@google.com on 12 Aug 2013 at 2:30

GoogleCodeExporter commented 9 years ago 
https://codereview.appspot.com/12786043/

Original comment by yan...@google.com on 12 Aug 2013 at 6:01

GoogleCodeExporter commented 9 years ago

Original comment by yan...@google.com on 14 Aug 2013 at 2:22