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

realpath error: The operation completed successfully #32298

Closed
vtjnash opened this issue Jun 11, 2019 · 5 comments · Fixed by #33116
Closed

realpath error: The operation completed successfully #32298

vtjnash opened this issue Jun 11, 2019 · 5 comments · Fixed by #33116
Labels
domain:filesystem Underlying file system and functions that use it system:windows Affects only Windows

Comments

@vtjnash
Copy link
Sponsor Member

vtjnash commented Jun 11, 2019 

julia> realpath("\\\\.\\C:")
ERROR: SystemError: realpath: The operation completed successfully.
Stacktrace:
 [1] #windowserror#45(::Nothing, ::typeof(Base.windowserror), ::Symbol, ::Bool) at .\error.jl:148
 [2] windowserror at .\error.jl:148 [inlined]
 [3] realpath(::String) at .\path.jl:351
 [4] top-level scope at REPL[39]:1

sure it did...

Julia Version 1.3.0-DEV.390
Commit c630477 (2019-06-11 10:36 UTC)
OS: Windows (i686-w64-mingw32)
@vtjnash vtjnash added the system:windows Affects only Windows label Jun 11, 2019
@GregPlowman
Copy link
Contributor

sure it did...

Ha Ha. Similar to mmap error on Windows:
ERROR: could not create file mapping: The operation completed successfully.

@musm
Copy link
Contributor

musm commented Jun 12, 2019 •
edited
Loading

Should we just use uv_fs_realpath instead of the custom windows implementation ? (I'm not sure if uv_fs_realpath fixes the issue, since I haven't tested it yet)

@vtjnash
Copy link
Sponsor Member Author

vtjnash commented Jul 15, 2019

yep, even if it doesn't fix the issue, it may make sense to switch (and/or contribute our implementation to libuv, if ours is better)

This was referenced Jul 16, 2019
Closed
Closed
@vtjnash
Copy link
Sponsor Member Author

vtjnash commented Aug 12, 2019

Looks like uv_fs_realpath is already very similar to ours, and could readily be enhanced to match ours

@vtjnash vtjnash added the domain:filesystem Underlying file system and functions that use it label Aug 12, 2019
@vtjnash
Copy link
Sponsor Member Author

vtjnash commented Aug 12, 2019

Looks like uv_fs_realpath is already very similar to ours, and could readily be enhanced to match ours

@musm musm mentioned this issue Aug 29, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
domain:filesystem Underlying file system and functions that use it system:windows Affects only Windows
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Replace realpath implementation with libuv musm/julia
3 participants
@vtjnash @musm @GregPlowman