Rockbox mail archiveSubject: 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
> You should be able to override version.sh in bootloader.make.
Override it with what? bootloader.make is used for all bootloaders, with
-- "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. KernighanReceived on 2009-07-05