This is the bug/patch tracker for Rockbox. Click here for more information.
Quick links: Bugs · Patches · Rockbox frontpage
FS#10517 - Add "Patches applied" to System -> Rockbox Info
Attached to Project:
Rockbox
Opened by Rosso Maltese (asettico) - Thursday, 13 August 2009, 00:02 GMT+1
Last edited by Jonathan Gordon (jdgordon) - Thursday, 05 August 2010, 15:13 GMT+1
Opened by Rosso Maltese (asettico) - Thursday, 13 August 2009, 00:02 GMT+1
Last edited by Jonathan Gordon (jdgordon) - Thursday, 05 August 2010, 15:13 GMT+1
|
DetailsThe patch adds an information line in the screen "Rockbox Info" about the patches applied.
The format is very simple: it shows the related FS numbers separated by commas, all in the same row; if it's longer than the LCD width, it scrolls only when selected, as usual. If no patches are applied, the line is not showed at all. The information is gathered at compile time, running make, and hard coded in the binary, so it grows the memory size just when some patch is found. Only the language and voice files always contain the strings used. It relies on the existence of files named $(ROOTDIR)/patchinfo/fsN.txt, where N is the related FS number. If a patch file contains also the differences that create this file, the algorithm is fed automagically. Just the file existence is checked, so empty files are valid. Since the information is stored inside the binary, it can be useful when trying different builds in the same installation, i.e. to investigate differences of the behaviour yields by some patch. A possible extension on which I'm thinking about is to use the patchinfo files content to list something more, like the FS summary. But in this case should be necessary to put each information one per line, so the height of the screen can grow enough to require the vertical scroll feature. Since as now I don't know how to manage it (probably is already done, I could add several stub lines, just to exceed the LCD height and see what's going on). This patch applies to r22278. Note to italian translators: ho aggiunto la correzione "s/Patch/pec/ig" in tools/voice.pl, ma non sono riuscito a trovare la combinazione giusta per la c dolce finale. Inoltre la e dovrebbe essere grave. |
This task depends upon
Closed by Jonathan Gordon (jdgordon)
Thursday, 05 August 2010, 15:13 GMT+1
Reason for closing: Rejected
Thursday, 05 August 2010, 15:13 GMT+1
Reason for closing: Rejected
http://www.rockbox.org/mail/archive/rockbox-archive-2008-02/0236.shtml
For instance, I wrote a sort of wrapper script that update, patch, build, and install. In the patch phase I could add instructions to create those patchinfo files.
Anyway, it was just an exercise, referring to the mail I indicated before.
I didn't manage patches outside FS, so always put them in it! ;-)