mattcasters commented on PR #7668: URL: https://github.com/apache/hop/pull/7668#issuecomment-5108806579
@hansva Agreed — landing jars in `lib/core` is the right model. The previous approach of moving Azure Key Vault to plugin-private `lib/` was a band-aid. The real issue was marketplace **skipping** all `lib/core/**` zip entries on install, while plugin assembly intentionally puts Maven `provided` deps there (shared system classpath). This PR now: 1. **Installs** `lib/core` jars from marketplace zips (with a log if content differs when replacing). 2. **Never removes** them on uninstall (sticky shared libs — once shared, they stay). 3. Keeps your #7649 azure `provided` packaging as-is. Branch was reset onto current `main` (clears the merge conflicts with #7657 etc.). Refcounted uninstall is a possible follow-up if we want to reclaim disk later. -- 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]
