Fix: use correct config directory path on: Android, Linux, Darwin, Windows

vlang / ui

A cross-platform UI library written in V

MIT License

2.31k stars 154 forks source link

Fix: use correct config directory path on: Android, Linux, Darwin, Windows #540

Closed sandbankdisperser closed 10 months ago

sandbankdisperser commented 11 months ago

This fixes the configuration paths for the different platforms:

Android: use the app data folder
Windows: use the AppData folder
Linux: Use $XDG_CONFIG_HOME folder from the xdg specification
Darwin: use the os.home_dir()
Default: use the os.home_dir() as fallback

This fixes the crash on android with the read-only filesystem error.

This PR will not work till the duplicate const issue from https://github.com/vlang/v/issues/19031 is fixed, so please keep it open till then.

sandbankdisperser commented 11 months ago

Should be ready for merge now

spytheman commented 10 months ago

I've rebased the PR over the current master, to avoid warnings and notices, and then used os.config_dir() to simplify the code a bit (it already does support the different platform specific folder locations, except for android ... it should have to be added too, in another PR to the main V repo, since code in os is critical for bootstrapping V).