Error Uploading Theme

[chrisbagnell]

Hello, I am trying to upload my first theme and getting a pretty long error. I am not sure where to start and how to fix it. It is using a lot of code from iVideo and I see no errors just by looking at the code. The theme works fine on my ipod classic. Any help is appreciated!

"Your wps didn't pass checkwps. Here's the output: Checking .rockbox/wps/ModernBox.wps... Couldn't open .rockbox/wps/ModernBox/BatteryStatus.bmp WPS parsing failure Checking .rockbox/wps/ModernBox.sbs... WPS parsed OK Checking .rockbox/wps/ModernBox.fms... WPS parsed OK Checking .rockbox/wps/ModernBox.wps... WPS parsing failure Error on line 37. %T(60,0,200,24,&quickscreen) ^ 'Parser callback returned error' Here Checking .rockbox/wps/ModernBox.sbs... WPS parsed OK Checking .rockbox/wps/ModernBox.fms... WPS parsing failure Error on line 37. %T(60,0,200,24,&cancel) ^ 'Parser callback returned error' Here Checking .rockbox/wps/ModernBox.wps... Couldn't open .rockbox/wps/ModernBox/BatteryStatus.bmp WPS parsing failure Checking .rockbox/wps/ModernBox.sbs... WPS parsed OK Checking .rockbox/wps/ModernBox.fms... WPS parsed OK Checking .rockbox/wps/ModernBox.wps... Couldn't open .rockbox/wps/ModernBox/BatteryStatus.bmp WPS parsing failure Checking .rockbox/wps/ModernBox.sbs... WPS parsed OK Checking .rockbox/wps/ModernBox.fms... WPS parsed OK Checking .rockbox/wps/ModernBox.wps... Couldn't open .rockbox/wps/ModernBox/BatteryStatus.bmp WPS parsing failure Checking .rockbox/wps/ModernBox.sbs... WPS parsed OK Checking .rockbox/wps/ModernBox.fms... WPS parsed OK Checking .rockbox/wps/ModernBox.wps... Couldn't open .rockbox/wps/ModernBox/BatteryStatus.bmp WPS parsing failure Checking .rockbox/wps/ModernBox.sbs... WPS parsed OK Checking .rockbox/wps/ModernBox.fms... WPS parsed OK Checking .rockbox/wps/ModernBox.wps... WPS parsing failure Error on line 37. %T(60,0,200,24,&quickscreen) ^ 'Parser callback returned error' Here Checking .rockbox/wps/ModernBox.sbs... WPS parsed OK Checking .rockbox/wps/ModernBox.fms... WPS parsing failure Error on line 37. %T(60,0,200,24,&cancel) ^ 'Parser callback returned error' Here Checking .rockbox/wps/ModernBox.wps... WPS parsing failure Error on line 37. %T(60,0,200,24,&quickscreen) ^ 'Parser callback returned error' Here Checking .rockbox/wps/ModernBox.sbs... WPS parsed OK Checking .rockbox/wps/ModernBox.fms... WPS parsing failure Error on line 37. %T(60,0,200,24,&cancel) ^ 'Parser callback returned error' Here Checking .rockbox/wps/ModernBox.wps... Couldn't open .rockbox/wps/ModernBox/BatteryStatus.bmp WPS parsing failure Checking .rockbox/wps/ModernBox.sbs... WPS parsed OK Checking .rockbox/wps/ModernBox.fms... WPS parsed OK Checking .rockbox/wps/ModernBox.wps... Couldn't open .rockbox/wps/ModernBox/BatteryStatus.bmp WPS parsing failure Checking .rockbox/wps/ModernBox.sbs... WPS parsed OK Checking .rockbox/wps/ModernBox.fms... WPS parsed OK Checking .rockbox/wps/ModernBox.wps... Couldn't open .rockbox/wps/ModernBox/BatteryStatus.bmp WPS parsing failure Checking .rockbox/wps/ModernBox.sbs... WPS parsed OK Checking .rockbox/wps/ModernBox.fms... WPS parsed OK Checking .rockbox/wps/ModernBox.wps... Couldn't open .rockbox/wps/ModernBox/BatteryStatus.bmp WPS parsing failure Checking .rockbox/wps/ModernBox.sbs... WPS parsed OK Checking .rockbox/wps/ModernBox.fms... WPS parsed OK Checking .rockbox/wps/ModernBox.wps... Couldn't open .rockbox/wps/ModernBox/BatteryStatus.bmp WPS parsing failure Checking .rockbox/wps/ModernBox.sbs... WPS parsed OK Checking .rockbox/wps/ModernBox.fms... WPS parsed OK Checking .rockbox/wps/ModernBox.wps... Couldn't open .rockbox/wps/ModernBox/BatteryStatus.bmp WPS parsing failure Checking .rockbox/wps/ModernBox.sbs... WPS parsed OK Checking .rockbox/wps/ModernBox.fms... WPS parsed OK Checking .rockbox/wps/ModernBox.wps... WPS parsing failure Error on line 70. %ax%Vl(currenta,120,70,190,18,2)%Vf(FFFFFF)%Vb(FFFFFF) ^ 'Parser callback returned error' Here Checking .rockbox/wps/ModernBox.sbs... WPS parsed OK Checking .rockbox/wps/ModernBox.fms... WPS parsing failure Error on line 67. %V(286,6,25,12,-)%Vf(b8b9bb)%Vb(FFFFFF) ^ 'Parser callback returned error' Here Checking .rockbox/wps/ModernBox.wps... Couldn't open .rockbox/wps/ModernBox/BatteryStatus.bmp WPS parsing failure"

[Dook]

As far as I'm aware, the issue here is the %T tags. They are tags that only work on touchscreen devices, since they require touch to perform the set action. The theme uploader is quite strict, so even if it runs on device and is entirely "legal" in a sense, it will reject the theme.

For now I'd say try removing these tags and see what happens. It is a little bit strange that it can't open BatteryStatus.bmp, If you have any further issues, consider uploading the code to github or an equiavaent and I'll take a look over it.

[Frankenpod]

Can't tell without looking at the entire theme, but sometimes you get problems because of case-sensitivity.  E.g. if the file is named batterystatus.bmp and the script tries to open BatteryStatus.bmp it will complain.  I _think_ that means even themes that run OK in the simulator on Windows (which is not case sensitive) can fail the syntax checker IIRC.

[Dook]

Can't tell without looking at the entire theme, but sometimes you get problems because of case-sensitivity.  E.g. if the file is named batterystatus.bmp and the script tries to open BatteryStatus.bmp it will complain.  I _think_ that means even themes that run OK in the simulator on Windows (which is not case sensitive) can fail the syntax checker IIRC.

Checking on linux at least, themes fail to load if the case doesn't match the file called. The error could is actually quite similar though to the theme uploader output chris provided, "Couldn't load '/.rockbox/wps/themify/Battery.bmp'".

[Frankenpod]

Can't tell without looking at the entire theme, but sometimes you get problems because of case-sensitivity.  E.g. if the file is named batterystatus.bmp and the script tries to open BatteryStatus.bmp it will complain.  I _think_ that means even themes that run OK in the simulator on Windows (which is not case sensitive) can fail the syntax checker IIRC.

Checking on linux at least, themes fail to load if the case doesn't match the file called. The error could is actually quite similar though to the theme uploader output chris provided, "Couldn't load '/.rockbox/wps/themify/Battery.bmp'".

That makes sense, as the issue is the different behaviour of Windows and Linux.  Making and testing themes (with the simulator) under Windows you don't actually see the case-sensitivity problems, I find...then you discover them with the theme checker when you upload them, as that clearly runs under linux.
Also IIRC if a theme had any touch-screen stuff left in it, it produced errors if you tried to upload it for a non-touchscreen target.

[Bilgus]

Inside rockbox we use case sensitive UNIX paths as well
and as the sim just kinda runs on top of the existing FS
I can see where that could bring out some issues

[chrisbagnell]

As far as I'm aware, the issue here is the %T tags. They are tags that only work on touchscreen devices, since they require touch to perform the set action. The theme uploader is quite strict, so even if it runs on device and is entirely "legal" in a sense, it will reject the theme.

For now I'd say try removing these tags and see what happens. It is a little bit strange that it can't open BatteryStatus.bmp, If you have any further issues, consider uploading the code to github or an equiavaent and I'll take a look over it.

Thank you, I have been burnt out from trying to figure out the error and it was BatteryStatus. the file name is "batteryStatus" and the wps file was the only one with an incorrect file name. Cheers!

[Dook]

Fantastic news! Was just checking out the theme and looks great, I remember seeing you post about this online before. Love that USB screen especially, keep up the great work!