[GitHub] [kafka] jthurne commented on pull request #10491: MINOR: Switch to using the Gradle RAT plugin
jthurne commented on pull request #10491: URL: https://github.com/apache/kafka/pull/10491#issuecomment-817901760 Interesting. When I run the rat task locally, those three files are identified (correctly) as binary files and RAT skips them. I'm not sure why they are being detected as text files on CI. But an easy workaround is to explicitly exclude that directory. I've pushed up a commit that does just that. -- 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
[GitHub] [kafka] jthurne commented on pull request #10491: MINOR: Switch to using the Gradle RAT plugin
jthurne commented on pull request #10491: URL: https://github.com/apache/kafka/pull/10491#issuecomment-817844704 > @jthurne This change causes new failures: Indeed. Unfortunately, I have been unable to reproduce them locally. One downside to the RAT plugin is that it doesn't output the files that fail the check to the build log. That makes this hard to diagnose since it seems to only happens on the CI server. My suspicion, however, is that this has something to do with file encoding. I'll keep digging. -- 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
[GitHub] [kafka] jthurne commented on pull request #10491: MINOR: Switch to using the Gradle RAT plugin
jthurne commented on pull request #10491: URL: https://github.com/apache/kafka/pull/10491#issuecomment-814950598 I believe the license failures were caused by a different change to trunk that has since been fixed. I merged the latest trunk HEAD into this PR to re-trigger CI. -- 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
[GitHub] [kafka] jthurne commented on pull request #10491: MINOR: Switch to using the Gradle RAT plugin
jthurne commented on pull request #10491: URL: https://github.com/apache/kafka/pull/10491#issuecomment-814916880 I'm looking into the build failures on this one. At first glance, the pr-merge build failed due to one of the test tasks crashing. Presumably that's not related at all to this change, which doesn't touch the test tasks. Some of the other failures site a RAT license check failure. I'm looking into this to make sure it isn't something new introduced by switching to the RAT plugin. -- 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
[GitHub] [kafka] jthurne commented on pull request #10491: MINOR: Switch to using the Gradle RAT plugin
jthurne commented on pull request #10491: URL: https://github.com/apache/kafka/pull/10491#issuecomment-814378302 Full disclosure: I'm an engineer that works for Gradle (on [Gradle Enterprise](https://gradle.com/), and I found these optimizations while using the project to test out Gradle Enterprise features. I put this change in a separate PR from [the other build optimizations](https://github.com/apache/kafka/pull/10490) since it is a slightly larger change, and since the HTML report name changes. -- 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
[GitHub] [kafka] jthurne commented on pull request #10491: MINOR: Switch to using the Gradle RAT plugin
jthurne commented on pull request #10491: URL: https://github.com/apache/kafka/pull/10491#issuecomment-814377383 @ijuma another one for you. -- 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