Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category User Interface
  • 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 asettico - 2009-08-12
Last edited by jdgordon - 2010-08-05

FS#10517 - Add "Patches applied" to System -> Rockbox Info

The 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.

Closed by  jdgordon
2010-08-05 13:13
Reason for closing:  Rejected

There is very little chance this will be accepted… We prefer people dont use custom builds, so if they do, they know which patchs they have added.. Also how is this supposed to work with patches that dont have a FS#?

OK, Daniel, I spent some effort on this, but it has been an interesting exercise! ;-) http://www.rockbox.org/mail/archive/rockbox-archive-2008-02/0236.shtml

You can always create “by hands” those files before running make.
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.

Ops, I misunderstood.
I didn’t manage patches outside FS, so always put them in it! ;-)

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing