jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [jclouds] nacx commented on pull request #78: Replace embedded and repackaged GSON library
Date Fri, 23 Oct 2020 08:46:45 GMT

nacx commented on pull request #78:
URL: https://github.com/apache/jclouds/pull/78#issuecomment-715197168


   So, the general idea of this is to move to the BND plugin (which I'm OK with) and to address
the issue of GSON unexported packages by exporting them from our jclouds-core module. That's
OK for me as well.
   
   jclouds needs to be able to upgrade to newer gson versions, and this is the cleanest approach
we've had so far to achieve that. If restructuring needed to accommodate the change imposes
some changes in downstream projects such as karaf-jclouds integration or Brooklyn, then let
it be and get those fixed as well. I think we have been very careful of not breaking stuff
and bringing everyone to the same page, but we can't be blocked forever on this and pinned
to the old gson library.
   
   @gurkerl83 has already stepped in and started to move forward with the modernization of
the karaf-jclouds integration, which is great. I'd suggest we move forward with this PR to
unblock jclouds, and anyone interested in having the Karaf and Brooklyn integration aligned
with newer jclouds versions to step in and help @gurkerl83 make that happen.
   


----------------------------------------------------------------
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:
users@infra.apache.org



Mime
View raw message