Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 5 Feb 2016 13:22:09 -0800
From:      NGie Cooper <yaneurabeya@gmail.com>
To:        Steven Hartland <smh@FreeBSD.org>
Cc:        src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-10@freebsd.org, FreeBSD Release Engineering Team <re@freebsd.org>
Subject:   Re: svn commit: r294981 - in stable/10/sys/boot: common efi efi/boot1 efi/include efi/libefi efi/loader efi/loader/arch/amd64 fdt i386/libi386
Message-ID:  <88C90200-1BC4-4C95-B37B-71AC33868217@gmail.com>
In-Reply-To: <201601281211.u0SCBhQG058154@repo.freebsd.org>
References:  <201601281211.u0SCBhQG058154@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help

> On Jan 28, 2016, at 04:11, Steven Hartland <smh@FreeBSD.org> wrote:
>=20
> Author: smh
> Date: Thu Jan 28 12:11:42 2016
> New Revision: 294981
> URL: https://svnweb.freebsd.org/changeset/base/294981
>=20
> Log:
>  MFC r281169, r293724, r293796, r294029, r294041, r294058
>=20
>  MFC r281169 (by andrew):
>  Make global variabled only used in this file static
>=20
>  MFC r294058:
>  Make common boot file_loadraw name parameter const
>=20
>  MFC r294041:
>  Remove unused reg param from fdt_fixup_memory
>=20
>  MFC r293724:
>  Enable warnings in EFI boot code
>=20
>  MFC r293796:
>  Fix typo in libefi.c
>=20
>  MFC r294029:
>  Only build EFI components on supported compilers
>=20
>  Sponsored by:	Multiplay

This commit broke tinderbox with arm:armeb on universe10a.freebsd.org:

cc1: warnings being treated as errors
In file included from =
/scratch/tmp/ngie/svn/sys/boot/arm/uboot/../../common/load_elf32.c:7:
/scratch/tmp/ngie/svn/sys/boot/arm/uboot/../../common/load_elf.c: In =
function 'elf32_parse_modmetadata':
/scratch/tmp/ngie/svn/sys/boot/arm/uboot/../../common/load_elf.c:859: =
warning: cast to pointer from integer of different size
*** [load_elf32.o] Error code 1

Given the issues that we have with ia64 not being able to build right =
now, I would consider pulling this out of ^/stable/10, fully fixing it, =
then reintegrating it.

Thanks,
-Ngie=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?88C90200-1BC4-4C95-B37B-71AC33868217>