olamy commented on a change in pull request #184: Repair Jenkins job - add a
full path to workspace when run on Windows
URL: https://github.com/apache/maven/pull/184#discussion_r221782663
##########
File path: Jenkinsfile
##########
@@ -79,7 +79,7 @@ for (String os in runITsOses) {
// on Windows, need a short path or we hit 256 character
limit for paths
// using EXECUTOR_NUMBER guarantees that concurrent builds
on same agent
// will not trample each other
- dir(isUnix() ? 'test' : "/mvn-it-${EXECUTOR_NUMBER}.tmp") {
+ dir(isUnix() ? 'test' :
"c:\\mvn-it-${EXECUTOR_NUMBER}.tmp") {
Review comment:
I'm not a windows expert but ``` c: ``` seems to be hardcoded and can cause
some issues?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
