-
-
Notifications
You must be signed in to change notification settings - Fork 600
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
Drive letter bug on win32 platforms in TSCache in typescript plugin #1133
Comments
Thanks for the issue. Please note the bit in the issue template about acceptable reproductions:
Please give https://git.io/fNzHA a read. The repo you've linked as your reproduction is definitely not minimal. We keep them simple as we're short on help and time is precious. |
Can confirm this. The TSCache is just replacing capital drive letters:
I will see if I can provide a minimal reproduction repo tomorrow. |
Also this doesn't seem to be limited to win32, had this issue on some win64 as well. Also seems to be related to the shell that's used. |
Reproduction Repository: https://github.com/timvahlbrock/repro-rollup-ts-lower-case-drive-letters Created #1141 to fix this. |
big thanks to @timvahlbrock and @lovettchris for both submitting PRs to fix this and for helping with this issue |
(note the lean4-infoview uses rollup)
Expected Behavior
Build succeeds on Linux, but fails on Windows.
Actual Behavior
Notice the drive letter included in the .rollup.cache path below, which makes for an invalid Windows path.
I have another pull request resolves one problem.
Which unblocks this problem.
Additional Information
The text was updated successfully, but these errors were encountered: