dweiss commented on code in PR #13164:
URL: https://github.com/apache/lucene/pull/13164#discussion_r1515605916


##########
gradle/testing/randomization.gradle:
##########
@@ -189,6 +201,14 @@ allprojects {
           systemProperty 'gradle.lib.dir', 
Paths.get(project.class.location.toURI()).parent.toAbsolutePath().toString().replace('\\',
 '/')
           systemProperty 'gradle.worker.jar', 
Paths.get("${gradleUserHome}/caches/${gradle.gradleVersion}/workerMain/gradle-worker.jar").toAbsolutePath().toString()
           systemProperty 'gradle.user.home', 
gradleUserHome.toPath().toAbsolutePath().toString()
+
+          
securityArgumentProvider.otherProperties.put("randomizedtesting.jar", 
project.provider { ->
+            return configurations.secManagerExclusions.resolve().find { 
it.name.startsWith("randomizedtesting-runner-") }.absolutePath

Review Comment:
   I tested with Windows, although without odd paths with spaces, etc... I 
believe property expansion is guaranteed to work properly in policy files but I 
can't find a spec reference for this. I'm not sure why these other properties 
use the complex logic to convert paths. 



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

Reply via email to