FS#12530 - Add a note and a warning about the location for recordings

Attached to Project: Rockbox
Opened by Alexander Levin (fml2) - Tuesday, 10 January 2012, 21:38 GMT
Task Type Patches
Category Manual
Status New
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Release 3.10
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Votes 0
Private No


The location for recordings can be set, but the only way to see the current value for the setting is to look into the config file. The patch adds a note stating that to the manual.

Also, there is a known bug ( FS#12517 ,  FS#12500 ) preventing saving the recorded files in the root directory (maybe this applies only to some targets, not all). The patch adds a warning about that.
This task depends upon

Comment by Nils Wallménius (nls) - Tuesday, 10 January 2012, 21:58 GMT
Isn't it better to fix the problems than documenting how things are broken?
Comment by Alexander Levin (fml2) - Wednesday, 11 January 2012, 07:01 GMT
Yes, it would be better, but as long as the error is still there, there should be a warning IMO since that can lead to the loss of a valuable recordings.
Comment by Thomas Martitz (kugel.) - Wednesday, 11 January 2012, 07:46 GMT
The fix is simple, just need to get around committing it.
Comment by Alexander Levin (fml2) - Wednesday, 11 January 2012, 10:32 GMT
The first part of the patch is still needed. I should have made two separate patches.
Comment by Dominik Riebeling (bluebrother) - Wednesday, 11 January 2012, 22:32 GMT
If such a problem exists it should be mentioned in the release notes, not the manual. If a problem exists in current builds it's not that much of a problem -- the development trunk can always be broken in one or the other way (yes, optimally such a bug should be fixed asap but that's a different story).
Comment by Alexander Levin (fml2) - Thursday, 12 January 2012, 20:49 GMT
I agree with you that the release notes are a better place for the warning. But the fact that the recording dir is not visible through the GUI is still there. It's not a bug, it's the way the GUI is designed (it's a bug in the GUI design if you will).