enapps-enorman commented on PR #34: URL: https://github.com/apache/sling-org-apache-sling-xss/pull/34#issuecomment-1618857169
Is it really still necessary to embed another copy of guava in this bundle? If the reason for embedding guava was that it required a newer version than was required by oak, then has that not been resolved with OAK-9989? Just referencing the oak-shaded-guava instead of shading and embedding another private copy again would make this bundle much smaller. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
