This is the bug/patch tracker for Rockbox. Click here for more information.
Quick links: Bugs · Patches · Rockbox frontpage
FS#6306 - Rename Tagcache to Database
Attached to Project:
Rockbox
Opened by Jonas Häggqvist (rasher) - Wednesday, 08 November 2006, 15:24 GMT+2
Last edited by Dominik Riebeling (bluebrother) - Wednesday, 08 November 2006, 17:26 GMT+2
Opened by Jonas Häggqvist (rasher) - Wednesday, 08 November 2006, 15:24 GMT+2
Last edited by Dominik Riebeling (bluebrother) - Wednesday, 08 November 2006, 17:26 GMT+2
|
DetailsAs 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.
tagcachetodatabase.diff
(15.4 KiB)
apps/tagcache.h | 10 ++--- apps/lang/english.lang | 22 +++++------ manual/configure_rockbox/file_view.tex | 12 +++--- manual/getting_started/installation.tex | 4 +- manual/rockbox_interface/main.tex | 6 +-- manual/rockbox_interface/playback.tex | 12 +++--- manual/rockbox_interface/tagcache.tex | 60 ++++++++++++++++---------------- tools/rockboxdev.sh | 2 - 8 files changed, 64 insertions(+), 64 deletions(-) |
This task depends upon
Closed by Jonas Häggqvist (rasher)
Saturday, 25 November 2006, 10:53 GMT+2
Reason for closing: Accepted
Additional comments about closing: Committed to cvs
Saturday, 25 November 2006, 10:53 GMT+2
Reason for closing: Accepted
Additional comments about closing: Committed to cvs
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)
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.
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/"
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.