Skip to content
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

Fix Windows problems #2665

Merged
merged 4 commits into from
Jan 19, 2016
Merged

Fix Windows problems #2665

merged 4 commits into from
Jan 19, 2016

Conversation

jonas054
Copy link
Collaborator

I did some testing on Windows (mingw) for the first time ever and tried to find solutions for the problems I encountered. The support for drive letter and the skipping of the symlink example are pretty straight forward. Then there's the strange behavior around backticks and temporary directories. Slightly more ugly solutions. See what you think.

Anyway, after these changes I was able to run bundle and rake in a Windows cmd window.

@alexdowad
Copy link
Contributor

Sweet!! I don't use MS products myself, but this is great for those who do!

@jawshooah
Copy link
Contributor

Appveyor is a great service for testing on Windows. Check out Overcommit's last build and their current config file for a sample.

@jonas054
Copy link
Collaborator Author

Not sure why the build is failing on jruby-9.0.1.0. I re-ran it, but it failed again.

@bbatsov
Copy link
Collaborator

bbatsov commented Jan 19, 2016

Looks like some Parser bug to me. //cc @whitequark

@whitequark
Copy link

I don't have time right now to chase bugs across RuboCop test suite and there is nothing obviously wrong that I've noticed, so I'm afraid you will have to debug that yourself.

@alexdowad
Copy link
Contributor

Nothing do do with parser. On JRuby + Windows, the rubocop command is not available in the shell after installing the gem.

I don't run Windows or I would help debug.

@bbatsov
Copy link
Collaborator

bbatsov commented Jan 19, 2016

Oh, yeah. I was looking at some parsing error messages a bit earlier in the output. But this build is still running on Linux...

@alexdowad
Copy link
Contributor

But this build is still running on Linux...

Ooh, right. I wonder if that bin/lsfiles.bat fix has somehow messed things up? Everything else looks innocuous...

@jonas054 jonas054 force-pushed the fix_windows_problems branch from f75952f to 175d0fb Compare January 19, 2016 10:32
@jonas054
Copy link
Collaborator Author

I wonder if that bin/lsfiles.bat fix has somehow messed things up?

That seems a likely candidate. I've removed that particular change, because it's only needed if you've got your heart set on running bundle in a Windows cmd window. (The important thing is that running rubocop works under Windows.)

Let's see how it goes...

@jonas054
Copy link
Collaborator Author

Yes. That did it.

@bbatsov
Copy link
Collaborator

bbatsov commented Jan 19, 2016

You'll have to rebase.

E.g. C:/Users/... on Windows.
We can't call FileUtils.ln_s on Windows.
Dir.mktmpdir called with a block fails when it tries to
delete the temporary directory if it's not empty.
@jonas054 jonas054 force-pushed the fix_windows_problems branch from 0c2a78c to 3c90d57 Compare January 19, 2016 11:30
@jonas054
Copy link
Collaborator Author

Rebased.

bbatsov added a commit that referenced this pull request Jan 19, 2016
@bbatsov bbatsov merged commit 9a8fe64 into rubocop:master Jan 19, 2016
@jonas054 jonas054 deleted the fix_windows_problems branch January 19, 2016 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants