diff options
Diffstat (limited to 'Tools/Source/GenBuild/org/tianocore/build/pcd/action/CollectPCDAction.java')
-rw-r--r-- | Tools/Source/GenBuild/org/tianocore/build/pcd/action/CollectPCDAction.java | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/Tools/Source/GenBuild/org/tianocore/build/pcd/action/CollectPCDAction.java b/Tools/Source/GenBuild/org/tianocore/build/pcd/action/CollectPCDAction.java index 9929005872..71a30b67ab 100644 --- a/Tools/Source/GenBuild/org/tianocore/build/pcd/action/CollectPCDAction.java +++ b/Tools/Source/GenBuild/org/tianocore/build/pcd/action/CollectPCDAction.java @@ -373,15 +373,13 @@ public class CollectPCDAction { if(Token.PCD_USAGE.UNKNOWN != token.isUsageInstanceExist(moduleName)) {
//
- // BUGBUG: It should *not* throw exception here. Becaues in MdePkg.fpd,
- // more than on BaseLib exist. But why? need confirmation.
+ // BUGBUG: It is legal that same base name exist in one FPD file. In furture
+ // we should use "Guid, Version, Package" and "Arch" to differ a module.
+ // So currently, warning should be disabled.
//
- //throw new EntityException(
- // "In module " + moduleName + " exist more than one PCD token " + token.cName
- // );
- ActionMessage.warning(this,
- "In module " + moduleName + " exist more than one PCD token " + token.cName
- );
+ //ActionMessage.warning(this,
+ // "In module " + moduleName + " exist more than one PCD token " + token.cName
+ // );
return null;
}
|