-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Failed to build binary package with gradlew.bat on Windows 7 (SP1, 64-bit) #60
Comments
What is the version of go command? What's happen if you run go command like this:
BTW, even in Win7/64bit, you can use the bin package which can be download from http://kobo.github.io/groovyserv/download.html. |
worked fine. No errors. I installed groovyserv with gvm but couldn't run it. That's why I tried building it afterwords. Here's the error:
|
There are two problems:
(1) could not install by setup scriptIt seems that you run Try to run the BTW, GroovyServ doesn't support Cygwin officially. So, it might be tough to use it on Cygwin even though you installed it successfully. (GroovyServ itself work even on Cygwin but it needs some cares, e.g. converting of cygpath of environment variables like JAVA_HOME.) Anyway, I'll improve the (2) could not build by gradleHmm. I don't know why the go command didn't work well... |
Thanks for looking into this.
|
Please run with
|
Looks like it doesn't know how to handle the space in "Program Files" |
...and also this
it appears that Windows firewall is blocking connection to port 1961 |
It seems the value of GROOVY_HOME has a white space. And Groovy can't work with it. |
You can also use the short path syntax like |
I'm still analyzing this issue... |
Could you paste this part of console log of
NOTE: I want to see only |
|
According to my today's analysis, only in the case of Windows, Go language's Command#Run() would fail both when the arguments have a blank and when the command path has a white-space, for some reason. And it shows the curious path in the error message, which is only the left side of white-space. I think that there are the workaround:
(a) is the easier and better way. |
I will try with short path format later today. As for (b), is it how to open up the port in Windows firewall (inbound rules)? |
Do you concern about this log?
This isn't due to firewall. It just said, "I couldn't connect to server." So, I think the core problem is the white-space in the command path. (It often causes problem in Windows.)
How about the result? something changed? |
It worked fine after replacing |
Great! About how to handle the path including white-spaces, it seems a problem of Go language, so far. I've created a ticket: https://code.google.com/p/go/issues/detail?id=9084 . If it wouldn't fix, I'll think a way of handling it in GroovyServ. For the meantime, please do it as the workaround. BTW, About "(2) could not build by gradle", do you want to solve it? |
yes, it would be nice to solve gradle build problem too. What do I need to do for that?
From the log:
|
About the issue of authtoken, it seems to be used an old groovyclient command. |
About "(2) could not build by gradle":
Expected:
|
That was it. After restarting groovyserver client was able to establish the connection. |
When running in cygwin console:
When running in cmd console:
For some reason it swallowed stdout and stderr |
Your Try this instead:
This will print stdout and stderr correctly:
|
Thank you! Thanks to your information, I've found the cause and another problem for debugging! The cause of the failure of building is a bad configuration only for Windows in And a thing to be difficult to analyze the issue is a way of handling System.out/err of a sub process executed from Gradle.
A direct use of
works in Mac and Linux, but it doesn't in Windows. There is no output from a sub process in Windows. So it makes a analysis be difficult. Now a workaround for you to build is just to comment out the L173 in build.gradle.
Then, you can build successfully, perhaps. |
The issue has just been closed. But the status is Now, I have to think about a workaround for this problem as GroovyServ... |
The text was updated successfully, but these errors were encountered: