search

aws / language-servers

AWS related Language Servers
Apache License 2.0
19 stars 18 forks source link

Update `patchYamlJsonPackage.js` to handle paths with spaces in `applyPatch` #258

Closed twwildey closed 3 months ago

twwildey commented 3 months ago

Problem

When language-servers is checked out under a directory that contains spaces in the full path, npm install will fail to complete due to patchYamlJsonPackage.js.

Solution

This commit fixes this issue by ensuring file paths are quoted to support spaces in them.

Testing

Ensured npm install succeeds in a directory with spaces in the full path (i.e. under a user's Application Support folder for Sublime Text on OS X).

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

twwildey commented 3 months ago

Created a new PR with a rebased commit: https://github.com/aws/language-servers/pull/268