Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Playlists
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by Jonas Häggqvist - 2006-11-08
Last edited by Dominik Riebeling - 2006-11-08

FS#6306 - Rename Tagcache to Database

As discussed on IRC, this patch renames tag cache to simply Database. Includes changes to manual and english.lang as well as changing the names of the files tagcache writes to disk. Pretty much untested, so give it a whirl.

Closed by  Jonas Häggqvist
2006-11-25 09:53
Reason for closing:  Accepted
Additional comments about closing:  

Committed to cvs

Dominik Riebeling commented on 2006-11-08 16:30

I think the attached patch qualifies this as a patch, not a bug ;-) Feel free to change it back if you disagree.

I think it would be better to rename it to “Tag Database” instead of simply “Database” – this doesn’t give a hint what is used to build the database, users might expect it to contain everything, including e.g. images, wave files etc. Additionally, I don’t think renaming the tagcache files itself is necessary – we can always argue that this is still the internal name when pointing a user to remove the tagcache files (and, in most cases this will only be “remove all *.tcd files, so the basename itself doesn’t matter that much)

Jonas Häggqvist commented on 2006-11-08 16:38

Thanks for fixing the wrong fields, I simply forgot to change them when submitting.

If we were to call it more than just database, I’d think “Music Database” was better. The reason why I moved the files were because they were mentioned in the manual (tagcache_changelog), but I agree it isn’t strictly necessary. But on the other hand, there are no real ill effects other than people having to rebuild.

Admin
Linus Nielsen Feltzing commented on 2006-11-08 19:01

I think “database” is the best, nice and simple.

Nils Wallménius commented on 2006-11-08 20:00

I vote for simply ‘database’ too.

rasher, I think you missed one here:
- \\item[ID3 Database:] This shows the files based on the \\setting{Tag Cache}

+ \\item[ID3 Database:] This shows the files based on the \\setting{Database}

Should probably be: (as the string changed in the .lang file)
(maybe it could be called something like “Database view”?)
- \\item[ID3 Database:] This shows the files based on the \\setting{Tag Cache}

+ \\item[Database:] This shows the files based on the \\setting{Database}

Also the change to rockboxdev.sh (the last part of the patch) was not supposed to
be there, right?
-prefix=”/usr/local” +prefix=”/home/rasher/rockbox/build-env/”

Jonas Häggqvist commented on 2006-11-09 13:23

nls: Right, right and right. Fixed.

I also fixed some occurrences of “The Database database” (caused by recklessly replacing “Tag Cache” with “Database”) and stopped treating database like a proper noun in most places (except when referring to the setting).

The only user-visible left is the configfile options, which are still called tagcache_*, but I’ll leave those.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing