dotnet / upgrade-assistant

A tool to assist developers in upgrading .NET Framework applications to .NET 6 and beyond
MIT License
1.11k stars 163 forks source link

Analyse using wrong path separator on mac #1574

Open mika76 opened 4 months ago

mika76 commented 4 months ago

Describe the bug

I ran upgrade assistant for a .net framework 4.8 project on a mac. I chose folder anlaysis and ran it. It said it generated the report and the result was...

We have gathered all required options and are ready to run analysis. Do you want to continue? [y/n] (y): y

Analyzing folder...

/Users/mladenmihajlovic/Documents/git/_blah/GF7 100%

Time elapsed 0 seconds

Discovered issues: 3
Discovered incidents: 566
Discovered story points: 566

  Mandatory  ███████████ 136
   Optional  10
  Potential  ███████████████████████████████████████████ 420
Information  0

Report with Unrestricted privacy mode saved at '/Users/mladenmihajlovic/Documents/git/_blah/GF7/report4/index.html'.
The file //Users/mladenmihajlovic/Documents/git/_blah/GF7/report4/index.html does not exist.

when I go into the generated folder all the files have the actual backslash character as part of their name so "index.html" is actually "\index.html" - if I try open the actual file in the browser is cannot open any other files because the backslash is makeing the filenames not match.

Screenshot 2024-06-23 at 11 49 54

To Reproduce

upgrade-assistant analyze and then choose folder. I ran it on a mac Sonoma 14.5 on a macbook 16" 2019 model with 16gb mem.

Further technical details

JensSchadron commented 4 months ago

Issue also occurs with Mac Sonoma 14.5 on macbook pro 14" 2021 (M1 Pro) with 32gb

almeidaalex commented 3 months ago

This bug was introduced in some new recent releases because I was using a previous version and this problem wasn't happening. Now I've got the same issue.

Include the output of upgrade-assistant --version 0.5.711.53382

Pit95 commented 1 month ago

Same on Fedora 40

Assistant version: 0.5.793.65096