ahgittin commented on pull request #78:
URL: https://github.com/apache/jclouds/pull/78#issuecomment-714696642
Seems to me like this PR is introducing two significant changes:
* use `bnd` instead of `jclouds.osgi.{im,ex}port` maven properties
* change how `gson` is imported/exported
If it's possible to have a PR for the first of these to be discussed, and
then a second for gson, that will make it much easier to consider.
The former (`bnd`) seems like a reasonable idea to me. It's messy either
way but slightly less messy.
The latter (`gson` change) is still confusing to me and as per the mailing
list my gut says we're better of not touching it unless we are fixing it
properly such as by moving to Jackson.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
