ruby-lsp icon indicating copy to clipboard operation
ruby-lsp copied to clipboard

Published 20 hours ago • verified publisher icon Shopify

vscode/asdf: search the default XDG base data directory

Open gi opened this issue 9 months ago • 1 comments

This updates the detection of the asdf data directory for setting the environment variable ASDF_DATA_DIR during activation.

Fixes #2023

Motivation

Fixes #2023

The XDG base directory specification is becoming more popular in outside of Linux. The default base data directory is ~/.local/share where applications store their user data within a directory under the base: e.g. ~/.local/share/asdf.

Implementation

This update includes the XDG base data directory in the list when detecting the location for activation.

Automated Tests

Yes.

Manual Tests

  1. Create the directory ~/.local/share/asdf/shims
  2. Start VS Code
  3. Open Ruby file

gi avatar May 13 '24 06:05 gi

(the rest of the team is at RubyKaigi so there might be a little delay until we ship this)

andyw8 avatar May 13 '24 17:05 andyw8

Just bumping this and wondering if the tests could be run so that this is ready to go when reviewed and if approved.

gi avatar May 20 '24 22:05 gi

Triggered the tests.

andyw8 avatar May 21 '24 14:05 andyw8

@gi I shipped a few improvements to the ASDF implementation which should fix many of the issues we were seeing before. I think this PR can just add the .local/share directory to the list and that should be good 👍.

vinistock avatar May 30 '24 19:05 vinistock

@vinistock I just tested the latest version, and the changes work on my machine. Thanks for updating. I'm going to close this as the changes are no longer needed.

The issue was fixed by https://github.com/Shopify/ruby-lsp/pull/2006

gi avatar May 30 '24 21:05 gi