adutra commented on PR #2964: URL: https://github.com/apache/polaris/pull/2964#issuecomment-3529485059
> Not sure whether we should add all the helm-building/testing dependencies as mandatory build requirements - at least not when running a ./gradlew check or test or so. > > OTOH it would be very convenient for many people to be able to run all those things in a simple way. > > WDYT about having a Dockerfile or docker-compose that includes all the necessary dependencies. Ensuring that writes to the source tree from those containers use the host's user is a bit tricky though. In this commit I implemented a check for missing dependencies that is entirely shell-based: https://github.com/apache/polaris/pull/2964/commits/09e638ed63b7b13b2bea0ae40e472a2967d6ce75 But then I removed it, thinking it was a bit overkill. But in hindsight, that was maybe better, wdyt? -- 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]
