-
Notifications
You must be signed in to change notification settings - Fork 79
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
Windows JRuby 64-bit Issues #26
Comments
Looks like GitHub was having issues when I submitted this, since it made 3. :) Closed the duplicates. This is the real deal. |
I'll look closer at this tomorrow, but for now try upgrading JRuby to
I'll join #vagrant tomorrow as well :) Den 19. jan. 2012 kl. 19:00 skrev Mitchell Hashimoto
|
And this only fails when using IO.pipe with JRuby on Windows. |
I was hoping the switch to the FFI implementation on Windows was a good first step in getting rid of the JRuby-specific code in this gem. If I can't figure out how to get the native fds, we should probably roll back to the old Unless of course @headius or @enebo has some ideas? Here's the reduced problem:
|
Would be really nice to have this fixed. This issue prevents us from updating to a newer Vagrant version right now which finally makes some feature usable on Windows. :) |
Absolutely would like to get this fixed. But this probably won't be stable if we roll back to ProcessBuilder either: https://github.com/jarib/childprocess/blob/master/spec/io_spec.rb#L81 The best solution would be to find some way to get the native fd. |
@denderello If you use RubyInstaller with 1.9.3 then Vagrant works swimmingly. I want the JRuby issues fixed too though :( |
Hi there, has this issue been forgotten? |
Not forgotten, I just have no idea how to solve it. Feel free to dig in to see if you can figure it out. |
I've done some research and this is what I've come up with so far:
I'm not really sure if this helps much but this is what I could come up with for now. BTW: I come from a Java background and have almost zero ruby knowledge/experience. |
Thanks for looking. The info about WINSOCK not being part of the file system is interesting and worth looking in to, but IIRC this is exactly where gave up the last time I tried (the FFI code is already special cased to avoid With ProcessBuilder I also had issues with pipes, which is why this spec is disabled (that's what I was trying to link to in my earlier comment). However we've since seen some patches in this area, so the situation may have improved. In any case I should make it so that JRuby+Windows users can choose between the FFI and the ProcessBuilder implementation. I'll have access to a Windows machine next week and hope to find some time to look into this then. If you want to try it out immediately, you could hack your Vagrant installation and replace the call to |
Thanks. This seem to do the trick for now. |
2 years later, I have to add to this: This error can also arise if you give Windows More subtly, this becomes an issue in the various shells of Windows, since So, if you're on Windows, you have to mindful to properly prepare paths so that they can be executed with |
I've just faced [possibly] the same issue and have one more thing to add to what @mitchellh said. The path must be not just existing, but also executable (which is not really clear in Windows environment). Calling a non-executable produces an error which does not really nail the issue (something like Just in case: I've hit it while trying to run ansible as vagrant provisioner in Windows. |
This issue re-appeared with JRuby 9.0.0.0pre1, getting the same error and |
Closing this due to age. Would welcome a pull request fixing the issue if it is still affecting people. Thanks! |
Hello hello,
Vagrant with JRuby on Windows 64-bit specifically has been seeing this error consistently:
See: hashicorp/vagrant#654
Note that the above user seems to have an international machine, but this happens on American machines as well, with the translation being "The operation completed successfully." but apparently did not?
The text was updated successfully, but these errors were encountered: