Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#5785 - Adding make install for devices

Attached to Project: Rockbox
Opened by Tom Meyer (tommeyer) - Wednesday, 09 August 2006, 22:02 GMT+2
Task Type Patches
Category Build environment
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Private No

Details

This patch modifies configue and buildzip.pl. Adds an option in configure, where the user can specify a mountpoint for the device, so that "make install" will copy files directly to the device, avoiding the multiple steps of make zip and then unzipping the file to the device.
(application/octet-stream)    patch (4.5 KiB)
This task depends upon

Closed by  Nicolas Pennequin (nicolas_p)
Wednesday, 21 February 2007, 01:50 GMT+2
Reason for closing:  Rejected
Comment by Daniel Stenberg (bagder) - Wednesday, 09 August 2006, 22:17 GMT+2
I disagree with the way its done in this patch.

The simulator already has a 'make install' target and if the "real" builds should have a similar I don't see any reason whatsoever to modify buildzip.pl. I find it better to keep that as simple as possible and add the extra cruft in the Makefile.

Also, I don't think the mount point is to be defined in the configure script since for many people the mount point can vary so it better be a variable you set at install time... Like "make DIR=/mnt/iriver install".
Comment by Tom Meyer (tommeyer) - Thursday, 10 August 2006, 00:18 GMT+2
Since buildzip.pl is the only thing that knows how to pull all the various pieces together into the .rockbox directory, it seemed like a logical place to put it. After all, it's basically replacing "zip" with "cp -r". I suppose that buildzip.pl might be a bad name for something that builds more things than zip, but I went with the simplest change to the existing codebase, rather than refactoring everything.

Regarding your second point, you can already say: make MOUNTPOINT=/mnt/iriver install, since it's already a variable in the Makefile.
Comment by Daniel Stenberg (bagder) - Thursday, 10 August 2006, 00:21 GMT+2
Let me stress my two points since it seems I wasn't clear enough;

1 - buildzip.pl doesn't need these changes since you can already accomplish this without changing it

2 - asking for the mount point in the configure is the wrong place to ask for such an information since building has nothing to do with mount points

Loading...