In this case, there should at least be a build tool option to break the build
if duplicates are found.
From: Tim Lewis [mailto:tim.le...@insyde.com]
Sent: Monday, August 25, 2014 5:04 PM
To: edk2-devel@lists.sourceforge.net
Subject: Re: [edk2] Duplicate PCDs in DSC files
Samer -
I would say that there are tool chains the depend on the last-instance winning.
Consider the case of #included .dsc files where the included .dsc has the
default value/storage type and the final resolution done later in the including
file.
Tim
From: El-Haj-Mahmoud, Samer [mailto:samer.el-haj-mahm...@hp.com]
Sent: Tuesday, August 26, 2014 1:59 AM
To: edk2-devel@lists.sourceforge.net<mailto:edk2-devel@lists.sourceforge.net>
Subject: [edk2] Duplicate PCDs in DSC files
It looks like the build tools will not warn you (or stop with an error) if
there are multiple declarations of the same PCD in the DSC file. In fact, the
same PCD could be initialized multiple times using different values and
different types (Fixed vs. Dynamic for instance), and the build tools will
silently pick the last instance of the PCD in the DSC File.
We just finished debugging a nasty issue caused by duplicate PCDs, and it would
have been nice to prevent this to begin with! Looks to me like a bug in the
tool.
Can we add the ability to break the build if there are duplicate PCD instances
in the DSC file?
Samer El-Haj-Mahmoud
System Firmware Architect
HP Servers
el...@hp.com<mailto:el...@hp.com>
T +1.281.514.5973
C +1.512.659.1523
Hewlett-Packard Company
hp.com/go/proliant/uefi<http://hp.com/go/proliant/uefi>
[Description: Description: C:\Users\elhajmah\HpLogo.png]
------------------------------------------------------------------------------
Slashdot TV.
Video for Nerds. Stuff that matters.
http://tv.slashdot.org/
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel
