breskeby commented on code in PR #14783: URL: https://github.com/apache/lucene/pull/14783#discussion_r2147569228
########## build-tools/build-infra/src/main/groovy/lucene.regenerate.icu.gradle: ########## @@ -30,14 +30,15 @@ def resources = rootProject.file("gradle/regenerate/icu") * download and compile a matching icu4c version automatically. */ -// Configure different icu4j dependencies. -configure(rootProject) { Review Comment: Declaring a configuration in one project and use it in another as done here before was one. of those things that had been technically possible forever, never recommended and now been deprecated. A good example of the problem with the historical approach in Gradle that in build logic there hasn't been any boundaries accross projects and the whole global model has been available from everywhere. -- 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: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org