Rockbox

Tasklist

FS#11924 - Use separate targets for RaaA devices

Attached to Project: Rockbox
Opened by Dave Chapman (linuxstb) - Wednesday, 09 February 2011, 00:46 GMT
Last edited by Dave Chapman (linuxstb) - Friday, 11 February 2011, 19:52 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 patch changes tools/configure to list the available RaaA targets in the main list of targets, instead of using a generic "Application" option with a follow-up question.

This also uses different target_id values for the different targets. The implication of this (I think) is that separate voice/lang files are required for each RaaA. IMO this is correct as they can potentially differ in the available features.

I'll commit this after it is confirmed I haven't broken anything - I don't have the Android or Maemo SDKs installed yet.
This task depends upon

Closed by  Dave Chapman (linuxstb)
Friday, 11 February 2011, 19:52 GMT
Reason for closing:  Accepted
Additional comments about closing:  v4 of patch committed as r29286
Comment by Thomas Jarosch (thomasjfox) - Wednesday, 09 February 2011, 08:39 GMT
Hello Dave,

took a quick look at it and it seems fine. I can give it a test run in the evening.

The "debian/rules" CONFIGURE_OPTIONS will break by this change. Should be easy to fix:
Add the pkgconfig call in there and determine the maemo version, too.

Cheers,
Thomas
Comment by Dave Chapman (linuxstb) - Wednesday, 09 February 2011, 20:02 GMT
Attached is an updated patch which fixes some issues found, including "make reconf" not working and maemo4cc being incomplete.

Comment by Dave Chapman (linuxstb) - Wednesday, 09 February 2011, 21:48 GMT
More fixes. Revert to the original maemocc function, passing the required maemo version as a parameter.
Comment by Dave Chapman (linuxstb) - Thursday, 10 February 2011, 12:37 GMT
This patch includes (untested) changes to debian/rules to choose the correct target based on the version of the Maemo SDK activated
Comment by Thomas Jarosch (thomasjfox) - Friday, 11 February 2011, 08:29 GMT
Hey Dave,

thanks for the updated patch, I'll give it a test run in the evening.

Thomas
Comment by Thomas Jarosch (thomasjfox) - Friday, 11 February 2011, 19:07 GMT
Patch v4 works fine, it builds the binary package out of the box.

I'd say commit it :)

Loading...