Rockbox

Tasklist

FS#11893 - wpsbuild.pl rewrite

Attached to Project: Rockbox
Opened by Marcin Bukat (MarcinBukat) - Thursday, 20 January 2011, 22:33 GMT
Last edited by Marcin Bukat (MarcinBukat) - Saturday, 04 June 2011, 21:48 GMT
Task Type Patches
Category Build environment
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Release 3.7.1
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

This is complete rewrite of wpsbuild.pl. The differences from old wpsbuild are:
1) Now it uses xml format for WPSLIST file (to make it more consistent) with help of XML::Simple CPAN module.
2) It scans all skin files for graphics needed (not only .wps)
3) Takes lcd dimensions parsing firmware/export/config/modelname.h file directly (instead of bizarre gcc preprocessing and regexp matching)
4) Should be easy to extend (for example to include rec skins if this ever happens)

At first I was thinking about fixing fms support in wpsbuild.pl but it was very hard to follow the code and writing something new from scratch was much easier.
This task depends upon

Closed by  Marcin Bukat (MarcinBukat)
Saturday, 04 June 2011, 21:48 GMT
Reason for closing:  Rejected
Additional comments about closing:  no going to be merged anytime soon
Comment by Marcin Bukat (MarcinBukat) - Friday, 21 January 2011, 21:34 GMT
1) wpsbuild2.pl - fix getlcdsizes() to parse correctly LCD defines if there is a comment in the same line
2) buildzip.pl - fix call to wpsbuild2.pl in verbose mode
Comment by Marcin Bukat (MarcinBukat) - Tuesday, 25 January 2011, 22:33 GMT
Changes compared to v2
wpsbuild2.pl
1) some code refactor
2) many small fixes
3) some sanity checks added
4) run with warnings turned on
5) handle all theme related configuration tags

wpsbuild2.pl/buildzip.pl
1) Handle classic_statusbar in wpsbuild2.pl (where this should be!)

WPSLIST.xml
1) reset backdrop to default if needed
2) explicitly state that cabbiev2 uses classic_statusbar.sbs

Loading...