Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#8447 - renames langs with latin chars

Attached to Project: Rockbox
Opened by harry tu (bookshare) - Saturday, 12 January 2008, 06:00 GMT+2
Last edited by Jonas Häggqvist (rasher) - Sunday, 27 July 2008, 18:54 GMT+2
Task Type Patches
Category Language
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

Localized lang names which used latin chars.
   renlang.zip (100.8 KiB)
This task depends upon

Closed by  Jonas Häggqvist (rasher)
Sunday, 27 July 2008, 18:54 GMT+2
Reason for closing:  Invalid
Additional comments about closing:  The patch included is not useful. More discussion on this subject in  FS#7637 
Comment by Peter D'Hoye (petur) - Sunday, 13 January 2008, 20:37 GMT+2
ehm... all I see in that patch are lines starting with '-'
what's the meaning of this?
Comment by harry tu (bookshare) - Sunday, 13 January 2008, 23:31 GMT+2
It is a zipped up patch. You need winzip or another similar tool.
Comment by Peter D'Hoye (petur) - Monday, 14 January 2008, 08:28 GMT+2
I'm not _that_ stupid ;)
I was talking about the patch inside....
Comment by harry tu (bookshare) - Tuesday, 15 January 2008, 05:06 GMT+2
Is this better? I did one more that I missed the first time around and redid the whole thing.
   lang.zip (126.9 KiB)
Comment by Peter D'Hoye (petur) - Tuesday, 15 January 2008, 08:18 GMT+2
can't open the zip...
Comment by Linus Nielsen Feltzing (linusnielsen) - Tuesday, 15 January 2008, 09:30 GMT+2
I don't understand this patch. All it does is to remove czech.lang, norsk.lang, portugues.lang, finnish.lang and espanol.lang.
Comment by Peter D'Hoye (petur) - Tuesday, 15 January 2008, 09:50 GMT+2
which was my point in my first comment.... ;)
Comment by harry tu (bookshare) - Wednesday, 16 January 2008, 01:59 GMT+2
It renames them to cestina.lang, castiliano.lang and all the other localised names.
Comment by Peter D'Hoye (petur) - Wednesday, 16 January 2008, 14:25 GMT+2
It does not... the patch only removes them
Comment by harry tu (bookshare) - Thursday, 17 January 2008, 03:42 GMT+2
What am I doing wrong? All I am typing in is svn rename apps/lang/czech.lang apps/lang/cestina.lang and so on for all the other languages. Then I do svn diff app/lang >lang.patch. What is wrong?
Comment by Peter D'Hoye (petur) - Thursday, 17 January 2008, 09:26 GMT+2
maybe it is best to just list the new names and not try to make a patch?
To make your patch, I think you need to add the renamed files as new ones and do a remove on the old ones.
Comment by Jonas Häggqvist (rasher) - Thursday, 17 January 2008, 17:15 GMT+2
Renaming the languages needs to be done with some consideration. If we simply rename and do nothing, users will end up with languages that never update (say some guy has set his language to finnish.lang, and updates go into suomi.lang). Would it perhaps be possible to break the current languages, such that they won't load and/or produce an error? Or perhaps truncate them so they don't include any strings.

For a transition period of course, after which the old ones should be deleted entirely.

Loading...