Re: [edk2] SVN r15336: somewhat ironically...

2014-03-24 Thread Li, Elvin
Hi Laszlo, The fix for build error is checked into 15386. Thanks Elvin -Original Message- From: Laszlo Ersek [mailto:ler...@redhat.com] Sent: Monday, March 24, 2014 9:24 PM To: edk2-devel@lists.sourceforge.net Subject: Re: [edk2] SVN r15336: somewhat ironically... On 03/24/14 05

Re: [edk2] [PATCH 2/3] MdeModulePkg: Implement EFI_FILE_PROTOCOL over Firmware Volumes

2014-03-24 Thread Tian, Feng
Thanks, Martin & Jackman I will review it and get back to you if I have feedback. Thanks Feng -Original Message- From: Olivier Martin [mailto:olivier.mar...@arm.com] Sent: Tuesday, March 25, 2014 04:23 To: Tian, Feng Cc: edk2-devel@lists.sourceforge.net; Brendan Jackman Subject: [PATCH

Re: [edk2] proposal to handle invalid compiler warnings

2014-03-24 Thread Bill Paul
Of all the gin joints in all the towns in all the world, Laszlo Ersek had to walk into mine at 06:32:57 on Monday 24 March 2014 and say: > On 03/23/14 03:19, Scott Duplichan wrote: > > Laszlo Ersek [mailto:ler...@redhat.com] wrote: > > > > ]Hence for gcc-4.4 to gcc-4.6, we should degrade -Wunini

[edk2] [PATCH 2/3] MdeModulePkg: Implement EFI_FILE_PROTOCOL over Firmware Volumes

2014-03-24 Thread Olivier Martin
From: Brendan Jackman This module implements Simple FileSystem protocol over Firmware Volume (FV). EFI Modules included into a FV can be listed and launched from the EFI Shell or any other EFI applications. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Brendan Jackman R

[edk2] [PATCH 1/3] Move BasePathLib from ShellPkg to MdeModulePkg

2014-03-24 Thread Olivier Martin
From: Brendan Jackman This library is useful to other modules than the shell - for example filesystem drivers. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Brendan Jackman Reviewed-by: Olivier Martin --- AppPkg/AppPkg.dsc| 2 +- Emul

[edk2] [PATCH 0/3] MdeModulePkg: Implement EFI_FILE_PROTOCOL over Firmware Volumes

2014-03-24 Thread Olivier Martin
please find the followed patches that expose the Firmware Volume (FV) as a Simple File System. EFI Modules included into a FV can be listed and launched from the EFI Shell or any other EFI applications. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Brendan Jackman Revie

[edk2] [PATCH 3/3] ArmPlatformPkg/ArmVExpressPkg: Add support for FV filesystems to ARM platforms

2014-03-24 Thread Olivier Martin
From: Brendan Jackman Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Brendan Jackman Reviewed-by: Olivier Martin --- ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-CTA15-A7.fdf| 3 +++ ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-CTA9x4.fdf | 2

[edk2] [PATCH] Ip4ConfigDxe: Look up 'Hostname' variable when using DHCP

2014-03-24 Thread Olivier Martin
From: Brendan Jackman This change adds the Hostname option to the DHCP Options. See section "3.14. Host Name Option" of RFC2132 "DHCP Options and BOOTP Vendor Extensions" The Hostname DHCP option if the UEFI variable "Hostname" is defined. This UEFI variable would contain the "Hostname" value.

Re: [edk2] proposal to handle invalid compiler warnings

2014-03-24 Thread Laszlo Ersek
On 03/23/14 03:19, Scott Duplichan wrote: > Laszlo Ersek [mailto:ler...@redhat.com] wrote: > ]Hence for gcc-4.4 to gcc-4.6, we should degrade -Wuninitialized from > ]error to warning. > > Has anyone proposed reducing the number of gcc versions supported? > Would dropping support for 4.4, 4,5 and

Re: [edk2] SVN r15336: somewhat ironically...

2014-03-24 Thread Laszlo Ersek
On 03/24/14 05:26, Li, Elvin wrote: > Hi Laszlo, > >> -Original Message- >> From: Laszlo Ersek [mailto:ler...@redhat.com] >> Sent: Sunday, March 23, 2014 4:15 AM >> To: edk2-devel list >> Subject: [edk2] SVN r15336: somewhat ironically... >> >> ... I'm heart-broken to report that the commit

Re: [edk2] [edk2-buildtools] proposal to handle invalid compiler warnings

2014-03-24 Thread Laszlo Ersek
On 03/23/14 02:22, Jordan Justen wrote: > For as long as I've been involved with edk2 (and Edk before that), > there has always been a strategy of: > 1. Ratchet up warnings as much as is tolerable > 2. Make warnings cause a build error > > I assume the goal is to try to keep the code base clean (f

Re: [edk2] proposal to handle invalid compiler warnings

2014-03-24 Thread Sergey Isakov
On 23.03.2014, at 6:19, Scott Duplichan wrote: > Laszlo Ersek [mailto:ler...@redhat.com] wrote: > > ]Hi, > ] > ]I'm interested in feedback for the following BuildTools-related idea. > ] > ]Over the past months working with OVMF I have distilled the following > ]conviction about invalid compiler

Re: [edk2] SVN r15336: somewhat ironically...

2014-03-24 Thread Sergey Isakov
Hi, This warning usually means that these codes are not complete. There is no action is SetVariable() returns not success. Just an ASSERT which is empty space in RELEASE compilation. Sergey On 23.03.2014, at 0:14, Laszlo Ersek wrote: > ... I'm heart-broken to report that the commit noted in the