summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--Tools/Source/GenBuild/org/tianocore/build/pcd/action/CollectPCDAction.java14
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;
}