Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Deprecated: Function create_function() is deprecated in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 104 Deprecated: The each() function is deprecated. This message will be suppressed on further calls in /sites/rockbox.org/flyspray/adodb/adodb.inc.php on line 845 Deprecated: Function create_function() is deprecated in /sites/rockbox.org/flyspray/includes/class.user.php on line 111 FS#8207 : buildzip.pl cleanup

Rockbox

Tasklist

FS#8207 - buildzip.pl cleanup

Attached to Project: Rockbox
Opened by Jonas Häggqvist (rasher) - Wednesday, 21 November 2007, 19:35 GMT
Last edited by Jonas Häggqvist (rasher) - Sunday, 20 July 2008, 22:39 GMT
Task Type Patches
Category Build environment
Status Closed
Assigned To Jonas Häggqvist (rasher)
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

This is my first stab at converting buildzip.pl to use native perl functions, rather than calling external apps (causing a pretty hefty slowdown on cygwin). So far, I've simply gone for the low-hanging fruit - simple cp and mv operations.

A recursive delete function should be fairly simple (to replace "rm -rf").

The next step is probably to write a sort of "find" implementation that takes a search expression and a callback function or something like that. It doesn't need to be a full find replacement of course. Just enough to do what buildzip needs (find and copy, I believe).

This will need a fair bit of testing of course, since we don't want the zips to be broken.
   buildzip-cleanup-fs8207.diff (4.5 KiB)
This task depends upon

Closed by  Jonas Häggqvist (rasher)
Sunday, 20 July 2008, 22:39 GMT
Reason for closing:  Accepted
Additional comments about closing:  Committed as r18108
Comment by Jonas Häggqvist (rasher) - Sunday, 20 July 2008, 21:11 GMT
Updated patch. Most things have been changed from shelling out to using Perl functionality.

Hopefully this provides a nice speedup.

I *think* this produces identical zip files to current SVN.
   buildzip-cleanup-fs8207-v2.di... (7.3 KiB)
Comment by Jonas Häggqvist (rasher) - Sunday, 20 July 2008, 22:10 GMT
Fixed a few issues:
- Cleanup .cp files
- Remove a leftover printf
- Actually create .rockbox/database.ignore
- Pipe all zip output to /dev/null
   buildzip-cleanup-fs8207-v3.di... (7.5 KiB)

Loading...