uschindler commented on a change in pull request #533:
URL: https://github.com/apache/lucene/pull/533#discussion_r770784718



##########
File path: gradle/java/folder-layout.gradle
##########
@@ -19,7 +19,7 @@
 allprojects {
   plugins.withType(JavaPlugin) {
     sourceSets {
-      main.java.srcDirs = ['src/java']
+      main.java.srcDirs = ['src/java' ]

Review comment:
       This change is a noop.

##########
File path: gradle/testing/randomization.gradle
##########
@@ -164,7 +164,9 @@ allprojects {
         // Enable security manager, if requested. We could move the selection 
of security manager and security policy
         // to each project's build/ configuration but it seems compact enough 
to keep it here for now.
         if 
(Boolean.parseBoolean(testOptionsResolved["tests.useSecurityManager"])) {
-          if (project.path == ":lucene:replicator") {
+          if (project.path == ":lucene:distribution-tests" || 
project.path.endsWith(".tests")) {

Review comment:
       distribution-tests module is now renamed to the name which matches by 
other boolean clause (endsWith).




-- 
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]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to