Skip to content

Windows paths need improved handling #32

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
alexcrichton opened this issue Jun 24, 2014 · 3 comments · Fixed by #56
Closed

Windows paths need improved handling #32

alexcrichton opened this issue Jun 24, 2014 · 3 comments · Fixed by #56
Labels
C-bug Category: bug

Comments

@alexcrichton
Copy link
Member

Due to the \ character, windows paths are both invalid URLs as well as causing all the tests to fail on windows. This can be closed when we no longer jankily replace \ with / and also when the tests pass on windows.

@alexcrichton
Copy link
Member Author

One blocker for this issue is dealing with rust-lang/rust#15149

@jhasse
Copy link

jhasse commented Jul 12, 2014

I'm still getting an error which seems related to \ on Windows:

~/git/rust-opengl-test (master=) cargo build
    Updating git repository `https://github.com/bjz/gl-rs.git`
fatal: '/home/jhasse/.cargo/git/checkouts/gl-rs-1669fe424cfaeccd/C:\msys64\home\jhasse\.cargo\git\db\gl-rs-1669fe424cfaeccd' does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
Unable to update https://github.com/bjz/gl-rs.git

To learn more, run the command again with --verbose.

~/git/rust-opengl-test (master=)

Should I open a new issue?

@alexcrichton
Copy link
Member Author

Yes please! A small repo to reproduce would also be much appreciated!

ehuss pushed a commit to ehuss/cargo that referenced this issue Nov 19, 2023
39: Add `cargo-fix` executable r=killercup a=oli-obk

fixes rust-lang#32
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
C-bug Category: bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants