search

mistakenelf / fm

A terminal based file manager
MIT License
564 stars 29 forks source link

Config file ignored on Windows #77

Closed zQueal closed 2 years ago

zQueal commented 2 years ago 
~/.config
❯ cat fm.yml
settings:
  borderless: true
  enable_logging: false
  enable_mousewheel: true
  pretty_markdown: true
  show_icons: false

Config is not loaded. Borders are still enabled, icons are enabled, etc.

mistakenelf commented 2 years ago

To be honest I have not tested in windows. Will take a look

mistakenelf commented 2 years ago

@zQueal is this on WSL in windows or are you running this actually inside windows?

zQueal commented 2 years ago

No, this is not in WSL. It's directly within the Windows terminal. I'm wondering if, under normal windows, the program doesn't look in %AppData%/fm.yml instead? (it doesn't)

A simple fix for this could be a --config flag which forces fm to use a specific configuration file.

mistakenelf commented 2 years ago

Alright I believe I have a solution. So on windows I wont create the config in ~/.config/fm it will just get created in the home directory which would be C:\Users\yourUserName\fm.yml in which you should be able to make changes and see them reflected in fm. As a side note, I am adding the ability to reload the config in app through the key command rc. Will cut a release later on today for you to try

mistakenelf commented 2 years ago

@zQueal released 0.12.0, let me know if that works for you

zQueal commented 2 years ago

Working perfectly! Thanks for the patch!

mistakenelf commented 2 years ago

Excellent, glad to hear it!