-
Notifications
You must be signed in to change notification settings - Fork 315
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
Could not scan deps with @ on Windows #2090
Comments
I'll try to reproduce the issue by scanning https://github.com/EricMCornelius/ease/ which depends on https://www.npmjs.com/package/@xtuc/long. |
At least on Linux I cannot reproduce the issue with the attached analyzer result file. Can you reproduce the issue on Windows with this file, @fb33? |
Possibly related ScanCode issues: aboutcode-org/scancode-toolkit#442, aboutcode-org/scancode-toolkit#542 |
Hi, I reproduce the issue with your file, here's the scan-result |
I'm working on Windows today, and there I can indeed reproduce the issue. However, it seems to be a pure ScanCode issue, as I can also reproduce it when calling ScanCode directly:
I've reported it upstream at aboutcode-org/scancode-toolkit#1876. |
Hopefully this PR will resolve this issue: aboutcode-org/scancode-toolkit#2310 |
I'm closing this in favor of the ScanCode version configuration / upgrade epic as this issue will disappear once ORT upgrades its default ScanCode version. |
Hello
I build ORT locally from the master branch
I run ORT on windows with git bash in admin mode.
The project analyzed is a node project.
(Note, the fix for non-existing github repo works fine, no extra curation needed)
After the scanner phase, I've got 205 errors like :
It seems to be a problem of encode/decode url.
The directory that is searched is C:\workspace\POCs\MOB_ORT\my-gateway-web\client-reports\downloads\NPM\0xtuc\long\4.2.2
But the directory that was created in windows file system is C:\workspace\POCs\MOB_ORT\my-gateway-web\client-reports\downloads\NPM%40xtuc\long\4.2.2\
Regards
The text was updated successfully, but these errors were encountered: