Rockbox

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

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#11825 - make target "Linkinstall" for developing

Attached to Project: Rockbox
Opened by Michael Stummvoll (Stummi) - Thursday, 23 December 2010, 12:13 GMT+2
Last edited by Michael Stummvoll (Stummi) - Tuesday, 28 December 2010, 11:48 GMT+2
Task Type Patches
Category Build environment
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Release 3.7.1
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

Hello Rockbox-Developers,

I added a make target called "linkinstall" which works like fullinstall, but symlinks the files instead of copying.

This is a good thing for developing with a simulator build. If you are working on a app, you can type "make linksinstall" after first build, which links the files to the .rockbox-directory in simdisk. Thus, after a change on an existing app, you only have to type "make", which builds the changed apps. The files in the simdisk-directory are direct affected now, so you dont have to type make install or fullinstall after this. This saves a lot of time.

Greetings,
Stummi

   linkinstall.patch (3.5 KiB) 
 tools/buildzip.pl |   27 +++++++++++++++++++++------
 tools/root.make   |    5 +++++
 2 files changed, 26 insertions(+), 6 deletions(-)

This task depends upon

Closed by  Michael Stummvoll (Stummi)
Tuesday, 28 December 2010, 11:48 GMT+2
Reason for closing:  Accepted
Comment by Frank Gevaerts (fg) - Thursday, 23 December 2010, 12:57 GMT+2
I think this is a useful idea.

The only think I'd personally do differently is the name. "link" means "hard link" to me, so I'd go for "symlinkinstall". Maybe that's just me though.

I didn't really review the implementation, but at first sight it looks fine to this non-perl person
Comment by Michael Stummvoll (Stummi) - Thursday, 23 December 2010, 13:11 GMT+2
Ok, changed this.
I also had to change the intention on make help output for this.

Greetings,
Michael
   linkinstall.patch (6.1 KiB) 
 tools/buildzip.pl |   27 +++++++++++++++++++++------
 tools/root.make   |   53 +++++++++++++++++++++++++++++------------------------
 2 files changed, 50 insertions(+), 30 deletions(-)

Loading...