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

Dotnet muxer doesn't respect Windows symlinks #83314

Closed
agocke opened this issue Mar 12, 2023 · 2 comments · Fixed by #99576
Closed

Dotnet muxer doesn't respect Windows symlinks #83314

agocke opened this issue Mar 12, 2023 · 2 comments · Fixed by #99576
Labels
area-Host in-pr There is an active PR which will close this issue when it is merged
Milestone

Comments

@agocke
Copy link
Member

agocke commented Mar 12, 2023

Scenario:

Install dotnet to a folder and create a symbolic link to dotnet.exe

Expected:

dotnet follows the symlink and resolves hostfxr etc as symlink-relative

Actual:

A fatal error occurred. The folder [C:\Users\andy\AppData\Local\dnvm\host\fxr] does not exist

@agocke agocke added this to AppModel Mar 12, 2023
@ghost ghost added the untriaged New issue has not been triaged by the area owner label Mar 12, 2023
@ghost
Copy link

ghost commented Mar 12, 2023

Tagging subscribers to this area: @vitek-karas, @agocke, @VSadov
See info in area-owners.md if you want to be subscribed.

Issue Details

Scenario:

Install dotnet to a folder and create a symbolic link to dotnet.exe

Expected:

dotnet follows the symlink and resolves hostfxr etc as symlink-relative

Actual:

A fatal error occurred. The folder [C:\Users\andy\AppData\Local\dnvm\host\fxr] does not exist

Author: agocke
Assignees: -
Labels:

area-Host

Milestone: -

@agocke agocke added this to the Future milestone Mar 12, 2023
@ghost ghost removed the untriaged New issue has not been triaged by the area owner label Mar 12, 2023
@elinor-fung
Copy link
Member

Related (would likely be the same solution):

pal::realpath is probably what would need to change:

// Return if path is valid and file exists, return true and adjust path as appropriate.
bool pal::realpath(string_t* path, bool skip_error_logging)

@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Jun 17, 2023
@agocke agocke closed this as completed in 3e572e9 Aug 2, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Sep 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-Host in-pr There is an active PR which will close this issue when it is merged
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants