sman-81 commented on code in PR #554:
URL: https://github.com/apache/maven-surefire/pull/554#discussion_r1059859610
##########
surefire-api/src/main/java/org/apache/maven/surefire/api/util/TempFileManager.java:
##########
@@ -180,6 +180,8 @@ public synchronized File createTempFile( String prefix,
String suffix )
throw new UncheckedIOException( new IOException(
"Unable to create temporary directory " +
tempDir.getAbsolutePath() ) );
}
+ // try to make temp file directory writable for all
+ tempDir.setWritable( true, false );
Review Comment:
I've removed the "writable for all" part. The chance of two similar user
names - working on the same machine! - resulting in identical sub directory
names is theoretic at best. Having the user name as part of the directory name
is helpful to quickly spot one's own directory. We are good leaving this part
of the code as-is IMO. I look forward to your feedback.
--
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...@maven.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
