Skip to content

Change install directory to ~/.wpilib or XDG_DATA_HOME/wpilib#841

Open
ThadHouse wants to merge 4 commits into
wpilibsuite:mainfrom
ThadHouse:installtodotfolder
Open

Change install directory to ~/.wpilib or XDG_DATA_HOME/wpilib#841
ThadHouse wants to merge 4 commits into
wpilibsuite:mainfrom
ThadHouse:installtodotfolder

Conversation

@ThadHouse

Copy link
Copy Markdown
Member

On linux, falls back to ~/.wpilib if XDG_DATA_HOME doesn't exist.

return homeRoot;

if (isLinux && xdgDataHome != null && !xdgDataHome.isBlank()) {
return new File(xdgDataHome, ".wpilib");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
return new File(xdgDataHome, ".wpilib");
return new File(xdgDataHome, "wpilib");

(As also mentioned in wpilibsuite/vscode-wpilib#885,) should be wpilib, not .wpilib. Making the folder double-hidden doesn't make sense (and probably violates the spec).

@sciencewhiz sciencewhiz changed the title Change install directory to ~/.wpilib or XDG_DATA_HOME/.wpilib Change install directory to ~/.wpilib or XDG_DATA_HOME/wpilib Jun 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants