Rockbox.org home
release
dev builds
extras
themes manual
wiki
device status forums
mailing lists
IRC bugs
patches
dev guide



Rockbox mail archive

Subject: Re: portalplayer bootloader versions

Re: portalplayer bootloader versions

From: Frank Gevaerts <frank_at_gevaerts.be>
Date: Sun, 5 Jul 2009 19:22:50 +0200

On Sun, Jul 05, 2009 at 07:18:38PM +0200, Thomas Martitz wrote:
> Jonas Häggqvist schrieb:
>> Frank Gevaerts wrote:
>>
>>
>>> Another way to do this would be to remove APPSVERSION from the
>>> bootloader makefile and set it in config-* instead.
>>>
>>
>> This seems most clear to me, at first glance - in combination with a flag
>> in show_logo.c whether to use BOOTLOADER_VERSION (or whatever is decided
>> to call it), or APPSVERSION (for svn bootloaders).
>>
>>
> I think config-* is the wrong place for version numbers in general. It
> should give info about what the hardware is capable to do, and some
> minor other target-dependant stuff (such as rockbox file extension).

I don't really like this either, but I don't really see many
alternatives

> You should be able to override version.sh in bootloader.make.

Override it with what? bootloader.make is used for all bootloaders, with
different versions.

Frank

-- 
"Debugging is twice as hard as writing the code in the first place.
Therefore, if you write the code as cleverly as possible, you are,
by definition, not smart enough to debug it." - Brian W. Kernighan
Received on 2009-07-05

Page was last modified "Jan 10 2012" The Rockbox Crew
aaa