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

Windows file links with whitespace not recognized in terminal #57139

Closed
hansmaad opened this issue Aug 24, 2018 · 1 comment
Closed

Windows file links with whitespace not recognized in terminal #57139

hansmaad opened this issue Aug 24, 2018 · 1 comment
Assignees
Labels
*duplicate Issue identified as a duplicate of another issue(s) terminal Integrated terminal issues upstream Issue identified as 'upstream' component related (exists outside of VS Code)

Comments

@hansmaad
Copy link

Issue Type: Bug

VS Code version: Code 1.26.1 (493869e, 2018-08-16T18:38:57.434Z)
OS version: Windows_NT x64 10.0.15063

System Info
Item Value
CPUs Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz (8 x 2904)
GPU Status 2d_canvas: enabled
checker_imaging: disabled_off
flash_3d: enabled
flash_stage3d: enabled
flash_stage3d_baseline: enabled
gpu_compositing: enabled
multiple_raster_threads: enabled_on
native_gpu_memory_buffers: disabled_software
rasterization: enabled
video_decode: enabled
video_encode: enabled
webgl: enabled
webgl2: enabled
Memory (System) 31.81GB (21.39GB free)
Process Argv C:\Users\1234\AppData\Local\Programs\Microsoft VS Code\Code.exe
Screen Reader no
VM 0%
Extensions (9)
Extension Author (truncated) Version
npm-intellisense chr 1.3.0
githistory don 0.4.2
tslint eg2 1.0.36
vscode-npm-script eg2 0.3.5
python ms- 2018.7.1
cpptools ms- 0.18.1
vsliveshare ms- 0.3.577
cmake twx 0.0.17
cmake-tools vec 1.1.1

Steps to reproduce:

  • Create a file c:\my folder\file.txt (the file must exist to be matched)
  • Open an integrated terminal (I tested with Git\\bin\\bash.exe)
  • type echo c:\my folder\file.txt

Expected behaviour:
c:\my folder\file.txt will be highlighted and can be clicked.

Actual behaviour:
Link is not highlighted. c:\myfolder\file.txt will work.

I found that the current _localLinkPattern for Windows is this

"((([a-zA-Z]:|\\.\\.?|\\~)|([^\\0<>\\?\\|\\/\\s!$`&*()\\[\\]+'\":;])+)?((\\\\|\\/)([^\\0<>\\?\\|\\/\\s!$`&*()\\[\\]+'\":;])+)+)"
                                              ^
----------------------------------------------^  that \s ??

I'm not sure if it makes any sense to match such file paths, but my code is currently under My Documents and my gulp output won't be recognized.

[gulp-typescript] C:\Data\xyz\My Documents\code\project\src\xcontroller.ts(179,24): error TS2554 E
@vscodebot vscodebot bot added the terminal Integrated terminal issues label Aug 24, 2018
@Tyriar
Copy link
Member

Tyriar commented Aug 24, 2018

Tracking in xtermjs/xterm.js#583

@Tyriar Tyriar closed this as completed Aug 24, 2018
@Tyriar Tyriar added *duplicate Issue identified as a duplicate of another issue(s) upstream Issue identified as 'upstream' component related (exists outside of VS Code) labels Aug 24, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Oct 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*duplicate Issue identified as a duplicate of another issue(s) terminal Integrated terminal issues upstream Issue identified as 'upstream' component related (exists outside of VS Code)
Projects
None yet
Development

No branches or pull requests

2 participants