Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Bugs
  • Category Plugins
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by G4Oblivion - 2011-12-24
Last edited by BigBambi - 2011-12-24

FS#12480 - disktidy: Sansa AMS MUSIC/ directory

I noticed that r31238 added the option of “cleaning” the MUSIC directory under “Sansa AMS”.
I do not believe that this should be an option for the following reasons:

1.) Although this directory is created automatically by the OF, the user could have easily created it themselves.
2.) “MUSIC” is a very common directory name; I use it for all my players.
3.) The OF uses the “MUSIC” directory to store your audio files, which are hardly “junk files”. This doesn’t make sense for users that use both the OF and rockbox.

Basically, it makes no sense to have this as an option. If the user want it they can add it to the disktidy config.

This was an unwelcomed surprise for me when I selected “ALL” like I usually do and find out all my audio files were deleted.

Closed by  BigBambi
2011-12-24 23:50
Reason for closing:  Not a Bug
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

Please bring it up on the developers mailing list.

I'm not sure if this really belongs here – it's not really a bug. The change is intentional and works as expected. And while I agree with your reasoning I think this should better be discussed on IRC / the mailing list instead of having it in the tracker.

It also might make sense to extend disktidy to allow hiding folders during the cleanup process, so a folder MUSIC can just get the hidden flag set in this case instead of removing it.

Isn't the commit about ##MUSIC# rather than MUSIC?

@bluebrother
I'm sorry, I wasn't sure where I should post this. I completely forgot about the mailing list.
IMO, just because a file/directory is automatically created by the OF, doesn't mean that it's a "junk file".
While the directory may be automatically created again, the contents of the directory are not. If the contents were just "junk files" then it would make sense to delete them, but the files are created by the user (e.g: MUSIC, AUDIOBOOKS, PODCASTS, etc.)

@kugel.
Both ##MUSIC# and MUSIC directories are options to be cleaned under < Sansa AMS > in disktidy. In this case I'm talking about the MUSIC directory.

Again, I'm very sorry for creating a flyspray about this. I should have placed it in the mailing list.

For the record, I agree with you - "Music" shouldn't be there. Having said that it isn't a bug as such as it is intended. I'll close it here, please bring it up on the -dev mailing list.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing