pyradio icon indicating copy to clipboard operation
pyradio copied to clipboard

[BUG] Recordings dir configuration not being respected

Open wandy-dev opened this issue 11 months ago • 3 comments

Describe the bug I'm seeing #241 and #253 on version 0.9.3.11.9

To Reproduce Steps to reproduce the behavior:

  1. Open the configuration menu inside pyradio
  2. Set recordings dir to something other than the default and save
  3. Quit pyradio
  4. Launch pyradio and you will see pyradio-recordings dir in the home dir, the config will still show your selected dir in the config menu
  5. Quit and relaunch again, you will see that pyradio has overwritten your configured recording dir value

Expected behavior Pyradio respects the recording dir configuration

System (please complete the following information):

  • OS: Fedora 41
  • Python version: 3.13.3
  • pipx version (if applicable): 1.7.1
  • pipx installation method: Unfortunately I can't remember this
  • PyRadio Version: 0.9.3.11.9
  • Player used: [e.g mpv, mplayer, vlc]

Additional context I built pyradio from source using sh devel/build_install_pyradio 3

wandy-dev avatar May 21 '25 14:05 wandy-dev

I am on v. 0.9.3.11.13 (the latest) and i cannot reproduce...

You are on v. 0.9.3.11.9 (which should not matter - i think), but just in case, can you please update to v. 0.9.3.11.13 and test again?

Just open a terminal and execute:

pyradio -U

s-n-g avatar May 21 '25 14:05 s-n-g

I've updated and I'm still seeing the issue 0.9.3.11.13

wandy-dev avatar Jun 03 '25 17:06 wandy-dev

Just to make sure...

After you set the new recording directory and pressing OK at the "Select Redordings Directory" window, returning to "PyRadio Configuration" window, do you press s to actually save the new config?

s-n-g avatar Jun 04 '25 12:06 s-n-g

The latest release will ask you to save the configuration file in case you try to close the config editor before saving. i really do hope this is the issue here.

I'm closing this issue for now. If you still have a problem, please do reopen it

s-n-g avatar Jun 16 '25 18:06 s-n-g