search

wpilibsuite / PathWeaver

Desktop application for generating motion paths
Other
65 stars 68 forks source link

Use wpilib/year/tools/PathWeaver for configuration folder #175

Open modelmat opened 4 years ago

modelmat commented 4 years ago

Is your feature request related to a problem? Please describe.
Currently, the PathWeaver folder is created in the $HOME directory, which gets cluterred on Linux.

Describe the solution you'd like
PathWeaver should store configuration files underneath ~/wpilib/<build-year>/tools/PathWeaver.

Additional context
https://github.com/wpilibsuite/PathWeaver/blob/eff54f5261ee9a8002e7d8fd182e66253997a1cc/src/main/java/edu/wpi/first/pathweaver/ProgramPreferences.java#L26 https://github.com/wpilibsuite/PathWeaver/blob/c5547bd265abdcc3cf6d2c0572b546985a523d67/src/main/java/edu/wpi/first/pathweaver/extensions/ExtensionManager.java#L22 https://github.com/wpilibsuite/PathWeaver/blob/c5547bd265abdcc3cf6d2c0572b546985a523d67/src/main/java/edu/wpi/first/pathweaver/Loggers.java#L40

gftabor commented 4 years ago

I am not opposed to the idea specifically. I am honestly surprised it isn't a hidden directory anyway (.Pathweaver instead of Pathweaver)

Even if Windows doesn't seem to honor the . means hidden convention.

carbotaniuman commented 4 years ago

We can't get to this in season, but it will definitely occur before 2021.