Rockbox

Tasklist

FS#10794 - .[r]sbs support for the theme site

Attached to Project: Rockbox
Opened by Dominik Wenger (Domonoky) - Wednesday, 18 November 2009, 22:16 GMT
Last edited by Dominik Wenger (Domonoky) - Wednesday, 02 December 2009, 20:29 GMT
Task Type Patches
Category Themes
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Release 3.4
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

This is a patch to add .[r]sbs file checking into the theme site.

Unfortnuatly i couldnt test this at moment. Maybe someone else can test this ?
This task depends upon

Closed by  Dominik Wenger (Domonoky)
Wednesday, 02 December 2009, 20:29 GMT
Reason for closing:  Accepted
Comment by Dominik Wenger (Domonoky) - Wednesday, 18 November 2009, 22:38 GMT
fix sps -> sbs typos.
Comment by Maurus Cuelenaere (mcuelenaere) - Thursday, 19 November 2009, 07:59 GMT
This looks OK, does it also run it through checkwps? (not sure if that one can check sbs'es (yet))
Comment by Thomas Martitz (kugel.) - Thursday, 19 November 2009, 17:21 GMT
the 3rd hunk looks wrong to me

I think checkwps can't check sbs yet, because it has hardcoded the extensions (wps, rwps).
Comment by Dominik Wenger (Domonoky) - Thursday, 19 November 2009, 17:51 GMT
Fix the error Kugel spottet.

And yes, this runs the sbs/rsbs files through checkwps. According to JdGordon checkwps should be able to cope with sbs file.

This still needs testing on some webserver to make sure it actually works :-)
Comment by Jonathan Gordon (jdgordon) - Thursday, 19 November 2009, 18:07 GMT
just qualidfy that 2nd sentance.... There is no difference (that i can immediatly think of) between .sbs and .wps apart from its extension... so it would surprise me if checkwps couldnt handle it...
Comment by Thomas Martitz (kugel.) - Thursday, 19 November 2009, 18:40 GMT
I also meant to mean that the hardcoded extensions is the only thing that needs to be fixed to make it parse sbs files :)
Comment by Maurus Cuelenaere (mcuelenaere) - Thursday, 19 November 2009, 19:15 GMT
You seem to add the filename to $rspsfound, but then check $rsbsfound.
Comment by Dominik Wenger (Domonoky) - Thursday, 19 November 2009, 20:04 GMT
ups. Fix typo
Comment by Dominik Wenger (Domonoky) - Wednesday, 02 December 2009, 19:55 GMT
Finally i was able to test this. And it works ! :-)

Loading...