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

GH-561 Fix file path handling for Windows #116

Merged
merged 1 commit into from
May 26, 2022
Merged

GH-561 Fix file path handling for Windows #116

merged 1 commit into from
May 26, 2022

Conversation

tortmayr
Copy link
Contributor

Update to Theia 1.25 to make use of the new Uri.path.fsPath() method which should ensure consistent file URI paths independent of the underlying
OS.

FIxes eclipse-glsp/glsp#561

Update to Theia 1.25 to make use of the new `Uri.path.fsPath()` method which should ensure consistent  file URI paths independent of the underlying 
OS.

FIxes eclipse-glsp/glsp#561
Copy link
Member

@planger planger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thank you!
However, I currently have to way to test this on windows. Works fine on Linux though! Thanks!

@MansarMalek
Copy link

Hello, thank you for your work
Does it also solve the problem with the theia json node minimal example? because the same error reproduces there

@tortmayr tortmayr merged commit e1dad52 into master May 26, 2022
tortmayr added a commit to eclipse-glsp/glsp-examples that referenced this pull request May 28, 2022
Consume a fix for OS-independent file path handling. (eclipse-glsp/glsp-theia-integration#116)
- Update glsp-theia-integration version
- Update to Theia 1.25

Part of eclipse-glsp/glsp#561
tortmayr added a commit to eclipse-glsp/glsp-examples that referenced this pull request May 29, 2022
Consume a fix for OS-independent file path handling. (eclipse-glsp/glsp-theia-integration#116)
- Update glsp-theia-integration version
- Update to Theia 1.25
- Update Readmes

Part of eclipse-glsp/glsp#561
tortmayr added a commit to eclipse-glsp/glsp-examples that referenced this pull request May 30, 2022
* #622 Update theia project templates

Consume a fix for OS-independent file path handling. (eclipse-glsp/glsp-theia-integration#116)
- Update glsp-theia-integration version
- Update to Theia 1.25
- Update Readmes

Part of eclipse-glsp/glsp#561

* Add sub-readmes and update tasks and launch config

* Align `TaskList` prefix in source files

Consistently use `TaskList` over `Tasklist` in all project templates


Co-authored-by: Philip Langer <planger@eclipsesource.com>
@tortmayr tortmayr deleted the gh-561 branch July 13, 2023 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Theia-integration does not work properly on windows
3 participants