svn commit: r293903 - in head/sys/boot: arm/uboot efi/fdt efi/libefi

[Steven Hartland]

On 14/01/2016 10:24, Andrew Turner wrote:
> On Thu, 14 Jan 2016 10:16:37 +0000
> Steven Hartland <steven at multiplay.co.uk> wrote:
>
>> On 14/01/2016 10:13, Andrew Turner wrote:
>>> On Thu, 14 Jan 2016 09:22:01 +0000 (UTC)
>>> Steven Hartland <smh at FreeBSD.org> wrote:
>>>   
>>>> Author: smh
>>>> Date: Thu Jan 14 09:22:01 2016
>>>> New Revision: 293903
>>>> URL: https://svnweb.freebsd.org/changeset/base/293903
>>>>
>>>> Log:
>>>>     Fix GCC warnings causing build failure after r293724
>>>>     
>>>>     Disable some compiler warnings for GCC (non-standard compiler)
>>>> fixing build failures introduced by r293724, which enabled WARNS in
>>>> the EFI boot code, when compiling with none standard compiler
>>>> (GCC).
>>>>     Raised by:	ian
>>>>     MFC after:	2 weeks
>>>>     X-MFC-With:	r293268
>>>>     Sponsored by:	Multiplay
>>> The attached would be a better (however untested) solution. Given we
>>> don't build boot1.efi or loader.efi with gcc it extends this to all
>>> the EFI code.
>>>
>>> Andrew
>> Not sure I like that idea as it would prevent people using EFI from
>> having a working system if they also choose GCC.
>>
> They already don't have loader.efi or boot1.efi with gcc, how will this
> patch break their existing system?
>
> Andrew
Ahh I see what you mean.

I see there's some discussion about this on IRC, if you can reply to 
this thread and let me know the conclusion, in case I miss it, I'll make 
the relevant changes :)

     Regards
     Steve