[ https://issues.apache.org/jira/browse/TINKERPOP-1493?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15566469#comment-15566469 ]
ASF GitHub Bot commented on TINKERPOP-1493: ------------------------------------------- GitHub user pauljackson opened a pull request: https://github.com/apache/tinkerpop/pull/456 TINKERPOP-1493 Groovy project doesn't build on Windows Removed support for user.dir property as it was being prepended to a fully qualified path and the second drive letter was making the path illegal. Made sure JarFile instances were being closed so that Groovy could delete the directory without encountering file locked errors. You can merge this pull request into a Git repository by running: $ git pull https://github.com/pauljackson/tinkerpop master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tinkerpop/pull/456.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #456 ---- commit de571c5a47138e4445e91bcf7897839ff6bf49f2 Author: PaulJackson123 <pauljackson...@verizon.net> Date: 2016-10-11T20:15:40Z TINKERPOP-1493 Groovy project doesn't build on Windows Removed support for user.dir property as it was being prepended to a fully qualified path and the second drive letter was making the path illegal. Made sure JarFile instances were being closed so that Groovy could delete the directory without encountering file locked errors. ---- > Groovy project doesn't build on Windows > --------------------------------------- > > Key: TINKERPOP-1493 > URL: https://issues.apache.org/jira/browse/TINKERPOP-1493 > Project: TinkerPop > Issue Type: Bug > Components: groovy > Affects Versions: 3.2.2 > Reporter: Paul Jackson > Priority: Minor > > Builds on Windows fail for two reasons. First the line to create extTestDir > is creating a path consisting of two full paths concatenated together. The > second drive letter is seen as an illegal character: > {code}private static final File extTestDir = new > File(System.getProperty("user.dir"), > TestHelper.makeTestDataDirectory(DependencyGrabberTest.class));{code} > Second, when it comes time to delete the directory it is locked. This is > because some instances of JarFile are created on it but not closed. -- This message was sent by Atlassian JIRA (v6.3.4#6332)