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.

Sign up for GitHub

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

Jump to bottom

Windows, cc_configure.bzl: do not use cygpath #2463

Closed
laszlocsomor opened this issue Feb 1, 2017 · 2 comments
Closed

Windows, cc_configure.bzl: do not use cygpath #2463

laszlocsomor opened this issue Feb 1, 2017 · 2 comments
Assignees
@laszlocsomor
Labels
P2 We'll consider working on this in future. (Assignee optional) type: feature request

Comments

@laszlocsomor
Copy link
Contributor

cc_configure.bzl performs a call to cygpath -m /, which only works if bazel runs under MSYS. This will not work when we cut the MSYS dependency (#2107).
@laszlocsomor laszlocsomor self-assigned this Feb 1, 2017
@laszlocsomor laszlocsomor added P2 We'll consider working on this in future. (Assignee optional) type: feature request labels Feb 1, 2017
@laszlocsomor
Copy link
Contributor Author

@meteorcloudy : FYI.
I have a fix almost ready.

@meteorcloudy
Copy link
Member

Ha, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@laszlocsomor laszlocsomor

Labels
P2 We'll consider working on this in future. (Assignee optional) type: feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@meteorcloudy @laszlocsomor