Rockbox

Tasklist

FS#11966 - Use separate modelnames and config files for RaaA

Attached to Project: Rockbox
Opened by Dave Chapman (linuxstb) - Thursday, 24 February 2011, 23:11 GMT
Last edited by Dave Chapman (linuxstb) - Sunday, 27 February 2011, 11:38 GMT
Task Type Patches
Category Build environment
Status Closed
Assigned To Dave Chapman (linuxstb)
Operating System Generic RaaA
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 patch is a continuation of r29286 to make RaaA builds more similar to regular targets.

It changes tools/configure to use a unique "modelname" for each target, instead of them all being called "application". It also splits the config/application.h file into separate files for each target, with the same name as $modelname.

tools/buildzip.pl checked for ($modelname eq 'application') so this has been changed to pass a --app parameter instead. When this is set to "yes", then buildzip.pl should behave the same as when $modelname was set to "application".

However, there seems to be a bug in my changes to buildzip.pl - "make zip" for an android build now fails, with wpsbuild.pl (called from buildzip.pl) trying to create a "/.rockbox" directory. I haven't had time to fix this yet - help welcome.

NOTE: Due to SVN not dealing with my use of "svn cp", you need to do the following before applying this patch:

cp firmware/export/config/application.h firmware/export/config/android.h
cp firmware/export/config/application.h firmware/export/config/nokian900.h
cp firmware/export/config/application.h firmware/export/config/nokian8xx.h
cp firmware/export/config/application.h firmware/export/config/sdlapp.h
This task depends upon

Closed by  Dave Chapman (linuxstb)
Sunday, 27 February 2011, 11:38 GMT
Reason for closing:  Accepted
Additional comments about closing:  Committed as r29418
Comment by Dave Chapman (linuxstb) - Friday, 25 February 2011, 08:36 GMT
Patch updated to r29236
Comment by Dave Chapman (linuxstb) - Friday, 25 February 2011, 23:27 GMT
I've tracked down the bug with buildzip.pl. It is caused by the line in current SVN which defines $app to be "($modelname eq "application"). This turns out to only be true for the *install: targets in root.make, as the other targets (e.g. zip) add backslashes to the modelname, meaning it contains the value "\"application\"". The attached patch fixes this by explicitly checking for $install when stripping the leading / - the only place outside the make_install() function where $app is checked.

Loading...