[Buildroot] [PATCH 3/7] boot/uboot: do not check hash for custom versions
Arnout Vandecappelle
arnout at mind.be
Fri May 1 13:14:30 UTC 2015
On 29/04/15 00:40, Yann E. MORIN wrote:
> For custom tarballs, it's quite obvious we can not check hashes.
>
> For custom versions, that's not so obvious. One might think we could
> have hashes for all the official releases, but that's not possible: if
> the user uses a released version of Buildroot (say 2015.05) much later
> in the future, and wants to use the then-released 2038.02 U-Boot. It now
> seems pretty obvious that Buildroot 2015.02 can't know the hash for
> U-Boot 2038.02.
>
> So, disable checking the hash for custom tarballs and custom versions.
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> Cc: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
> Cc: Vincent Stehle <vincent.stehle at laposte.net>
> ---
> boot/uboot/uboot.mk | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/boot/uboot/uboot.mk b/boot/uboot/uboot.mk
> index 01aaf2c..410d8d8 100644
> --- a/boot/uboot/uboot.mk
> +++ b/boot/uboot/uboot.mk
> @@ -17,6 +17,7 @@ ifeq ($(UBOOT_VERSION),custom)
> UBOOT_TARBALL = $(call qstrip,$(BR2_TARGET_UBOOT_CUSTOM_TARBALL_LOCATION))
> UBOOT_SITE = $(patsubst %/,%,$(dir $(UBOOT_TARBALL)))
> UBOOT_SOURCE = $(notdir $(UBOOT_TARBALL))
> +BR_NO_CHECK_HASH_FOR += $(UBOOT_SOURCE)
> else ifeq ($(BR2_TARGET_UBOOT_CUSTOM_GIT),y)
> UBOOT_SITE = $(call qstrip,$(BR2_TARGET_UBOOT_CUSTOM_REPO_URL))
> UBOOT_SITE_METHOD = git
> @@ -27,6 +28,9 @@ else
> # Handle stable official U-Boot versions
> UBOOT_SITE = ftp://ftp.denx.de/pub/u-boot
> UBOOT_SOURCE = u-boot-$(UBOOT_VERSION).tar.bz2
> +ifeq ($(BR2_TARGET_UBOOT_CUSTOM_VERSION),y)
> +BR_NO_CHECK_HASH_FOR += $(UBOOT_SOURCE)
> +endif
> endif
Since we really only have a hash for the latest version, I would prefer a
simple check after the entire condition:
ifeq ($(BR2_TARGET_UBOOT_LATEST_VERSION),)
BR_NO_CHECK_HASH_FOR += $(UBOOT_SOURCE)
endif
This redundantly handles the VCS cases, but OTOH it doesn't hurt to have these
tarballs in there either. Actually, before patch 2/7, the VCS cases wouldn't
even be added because UBOOT_SOURCE is not set yet at that point :-)
Regards,
Arnout
>
> ifeq ($(BR2_TARGET_UBOOT_FORMAT_ELF),y)
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
More information about the buildroot
mailing list