[GitHub] incubator-rya issue #264: RYA-447 Fixed issue building rya.shell under Windo...

2018-01-30 Thread ejwhite922
Github user ejwhite922 commented on the issue: https://github.com/apache/incubator-rya/pull/264 @kchilton2 do you have any preference on the suggested solutions? ---

[GitHub] incubator-rya issue #264: RYA-447 Fixed issue building rya.shell under Windo...

2018-01-22 Thread ejwhite922
Github user ejwhite922 commented on the issue: https://github.com/apache/incubator-rya/pull/264 System.getProperty("user.home") returns paths with backslashes in Windows (i.e. "C:\\Users\\" is returned). The parsing of this fails in a call to String.replaceFirst(String regex, String

[GitHub] incubator-rya issue #264: RYA-447 Fixed issue building rya.shell under Windo...

2018-01-22 Thread kchilton2
Github user kchilton2 commented on the issue: https://github.com/apache/incubator-rya/pull/264 What does System.getProperty("user.home") look like when you're running on Windows? If it has forward slashes, then the test should also use forward slashes since you will be providing a

[GitHub] incubator-rya issue #264: RYA-447 Fixed issue building rya.shell under Windo...

2018-01-19 Thread ejwhite922
Github user ejwhite922 commented on the issue: https://github.com/apache/incubator-rya/pull/264 @kchilton2 @pujav65 Both the test and shell command fail in Windows since it will return a path for rootedFile without file separators. Another option is we could set the path back

[GitHub] incubator-rya issue #264: RYA-447 Fixed issue building rya.shell under Windo...

2018-01-19 Thread pujav65
Github user pujav65 commented on the issue: https://github.com/apache/incubator-rya/pull/264 I like what Eric did. Is the issue that the test is failing, or that this shell command just doesn't work in windows? If the latter, then Eric's approach seems fine. If the former, then

[GitHub] incubator-rya issue #264: RYA-447 Fixed issue building rya.shell under Windo...

2018-01-18 Thread ejwhite922
Github user ejwhite922 commented on the issue: https://github.com/apache/incubator-rya/pull/264 An alternative could be to escape all '\\' characters instead of changing to '/'. Would this be a better solution? final String userHome = System.getProperty("user.home"); //

[GitHub] incubator-rya issue #264: RYA-447 Fixed issue building rya.shell under Windo...

2018-01-16 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/incubator-rya/pull/264 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/668/ ---

[GitHub] incubator-rya issue #264: RYA-447 Fixed issue building rya.shell under Windo...

2018-01-16 Thread ejwhite922
Github user ejwhite922 commented on the issue: https://github.com/apache/incubator-rya/pull/264 asfbot build ---

[GitHub] incubator-rya issue #264: RYA-447 Fixed issue building rya.shell under Windo...

2018-01-16 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/incubator-rya/pull/264 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/664/Failed Tests: