summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorlhauch <lhauch@6f19259b-4bc3-4df7-8a09-765794883524>2006-08-14 18:41:39 +0000
committerlhauch <lhauch@6f19259b-4bc3-4df7-8a09-765794883524>2006-08-14 18:41:39 +0000
commit391dbbb1c00daefe78e7e44499d048943ca866ae (patch)
tree2c69342e331e865188c449c7f50ad17fa4c84f40
parent3f7b510edbe3c10b533f36f490e591782d14e929 (diff)
downloadedk2-391dbbb1c00daefe78e7e44499d048943ca866ae.zip
edk2-391dbbb1c00daefe78e7e44499d048943ca866ae.tar.gz
edk2-391dbbb1c00daefe78e7e44499d048943ca866ae.tar.bz2
Fixed grammar in messages.
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@1256 6f19259b-4bc3-4df7-8a09-765794883524
-rw-r--r--Tools/Source/GenBuild/org/tianocore/build/FfsProcess.java2
-rw-r--r--Tools/Source/GenBuild/org/tianocore/build/FrameworkBuildTask.java18
-rw-r--r--Tools/Source/GenBuild/org/tianocore/build/GenBuildTask.java14
-rw-r--r--Tools/Source/GenBuild/org/tianocore/build/ModuleBuildFileGenerator.java6
-rw-r--r--Tools/Source/GenBuild/org/tianocore/build/autogen/AutoGen.java20
-rw-r--r--Tools/Source/GenBuild/org/tianocore/build/autogen/AutogenLibOrder.java6
-rw-r--r--Tools/Source/GenBuild/org/tianocore/build/autogen/CommonDefinition.java4
-rw-r--r--Tools/Source/GenBuild/org/tianocore/build/fpd/FpdParserTask.java8
-rw-r--r--Tools/Source/GenBuild/org/tianocore/build/fpd/PlatformBuildFileGenerator.java12
-rw-r--r--Tools/Source/GenBuild/org/tianocore/build/global/GlobalData.java28
-rw-r--r--Tools/Source/GenBuild/org/tianocore/build/global/OnDependency.java2
-rw-r--r--Tools/Source/GenBuild/org/tianocore/build/global/Spd.java8
-rw-r--r--Tools/Source/GenBuild/org/tianocore/build/global/VariableTask.java2
-rw-r--r--Tools/Source/GenBuild/org/tianocore/build/pcd/action/PCDAutoGenAction.java8
-rw-r--r--Tools/Source/GenBuild/org/tianocore/build/pcd/action/PcdDatabase.java12
-rw-r--r--Tools/Source/GenBuild/org/tianocore/build/pcd/action/PlatformPcdPreprocessActionForBuilding.java12
-rw-r--r--Tools/Source/GenBuild/org/tianocore/build/toolchain/ConfigReader.java4
17 files changed, 83 insertions, 83 deletions
diff --git a/Tools/Source/GenBuild/org/tianocore/build/FfsProcess.java b/Tools/Source/GenBuild/org/tianocore/build/FfsProcess.java
index 5541be4..15f9759 100644
--- a/Tools/Source/GenBuild/org/tianocore/build/FfsProcess.java
+++ b/Tools/Source/GenBuild/org/tianocore/build/FfsProcess.java
@@ -164,7 +164,7 @@ public class FfsProcess {
System.out.println("Warning: this module doesn't specify a FfsFormatKey. ");
}
else {
- throw new BuildException("Can't find FfsFormatKey [" + buildType + "] in FPD file. ");
+ throw new BuildException("Can't find the FfsFormatKey [" + buildType + "] attribute in the FPD file!");
}
if (ffs == null) {
diff --git a/Tools/Source/GenBuild/org/tianocore/build/FrameworkBuildTask.java b/Tools/Source/GenBuild/org/tianocore/build/FrameworkBuildTask.java
index b2c3e15..2617f06 100644
--- a/Tools/Source/GenBuild/org/tianocore/build/FrameworkBuildTask.java
+++ b/Tools/Source/GenBuild/org/tianocore/build/FrameworkBuildTask.java
@@ -112,10 +112,10 @@ public class FrameworkBuildTask extends Task{
//
File buildFile = null;
if (msaFiles.size() > 1) {
- throw new BuildException("More than one MSA file under current directory. It is not allowd. ");
+ throw new BuildException("Having more than one MSA file in a directory is not allowed!");
}
else if (msaFiles.size() == 1 && activePlatform == null) {
- throw new BuildException("If try to build a single module, please set ACTIVE_PLATFORM in file [Tool/Conf/target.txt]. ");
+ throw new BuildException("If trying to build a single module, please set ACTIVE_PLATFORM in file [Tool/Conf/target.txt]. ");
}
else if (msaFiles.size() == 1 && activePlatform != null) {
//
@@ -136,14 +136,14 @@ public class FrameworkBuildTask extends Task{
// If there is no build files or FPD files or MSA files, stop build
//
else {
- throw new BuildException("Can't find any FPD files or MSA files in current directory. ");
+ throw new BuildException("Can't find any FPD or MSA files in the current directory. ");
}
//
// Build every FPD files (PLATFORM build)
//
if (buildFile.getName().endsWith(".fpd")) {
- System.out.println("Start to build FPD file [" + buildFile.getPath() + "] ..>> ");
+ System.out.println("Processing the FPD file [" + buildFile.getPath() + "] ..>> ");
FpdParserTask fpdParserTask = new FpdParserTask();
fpdParserTask.setType(type);
fpdParserTask.setProject(getProject());
@@ -156,8 +156,8 @@ public class FrameworkBuildTask extends Task{
//
else if (buildFile.getName().endsWith(".msa")) {
File tmpFile = new File(GlobalData.getWorkspacePath() + File.separatorChar + activePlatform);
- System.out.println("Using FPD file [" + tmpFile.getPath() + "] as active platform. ");
- System.out.println("Start to build MSA file [" + buildFile.getPath() + "] ..>> ");
+ System.out.println("Using the FPD file [" + tmpFile.getPath() + "] for the active platform. ");
+ System.out.println("Processing the MSA file [" + buildFile.getPath() + "] ..>> ");
GenBuildTask genBuildTask = new GenBuildTask();
genBuildTask.setSingleModuleBuild(true);
genBuildTask.setType(type);
@@ -204,13 +204,13 @@ public class FrameworkBuildTask extends Task{
allFiles[index] = iter.next();
index++;
}
- System.out.println("Find " + allFiles.length + " FPD and MSA files: ");
+ System.out.println("Finding " + allFiles.length + " FPD and MSA files: ");
for (int i = 0; i < allFiles.length; i++) {
System.out.println("[" + (i + 1) + "]: " + allFiles[i].getName());
}
boolean flag = true;
- System.out.print("Please select one file to build:[1] ");
+ System.out.print("Please select one of the following FPD files to build:[1] ");
do{
BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
try {
@@ -293,7 +293,7 @@ public class FrameworkBuildTask extends Task{
str = getValue("ACTIVE_PLATFORM", targetFileInfo);
if (str != null && ! str.trim().equals("")) {
if ( ! str.endsWith(".fpd")) {
- throw new BuildException("FPD file's file extension must be \".fpd\"");
+ throw new BuildException("FPD file's extension must be \".fpd\"!");
}
activePlatform = str;
}
diff --git a/Tools/Source/GenBuild/org/tianocore/build/GenBuildTask.java b/Tools/Source/GenBuild/org/tianocore/build/GenBuildTask.java
index 7a9d7c4..cdccfa5 100644
--- a/Tools/Source/GenBuild/org/tianocore/build/GenBuildTask.java
+++ b/Tools/Source/GenBuild/org/tianocore/build/GenBuildTask.java
@@ -144,7 +144,7 @@ public class GenBuildTask extends Ant {
String packageGuid = getProject().getProperty("PACKAGE_GUID");
String packageVersion = getProject().getProperty("PACKAGE_VERSION");
if (moduleGuid == null || packageGuid == null) {
- throw new BuildException("GenBuild parameters error. ");
+ throw new BuildException("GenBuild parameter error.");
}
PackageIdentification packageId = new PackageIdentification(packageGuid, packageVersion);
moduleId = new ModuleIdentification(moduleGuid, moduleVersion);
@@ -225,7 +225,7 @@ public class GenBuildTask extends Ant {
if (moduleSupportedArchs != null) {
for (int k = 0; k < archList.length; k++) {
if ( ! moduleSupportedArchs.contains(archList[k])) {
- throw new BuildException("ARCH [" + archList[k] + "] is not supported by " + moduleId + ". " + moduleId + " only supports [" + moduleSupportedArchs + "].");
+ throw new BuildException("Specified architecture [" + archList[k] + "] is not supported by " + moduleId + ". The module " + moduleId + " only supports [" + moduleSupportedArchs + "] architectures.");
}
}
}
@@ -240,7 +240,7 @@ public class GenBuildTask extends Ant {
// Whether the module is built before
//
if (moduleId.isLibrary() == false && GlobalData.hasFpdModuleSA(fpdModuleId) == false) {
- System.out.println("\nWARNING: " + moduleId + " for " + archList[k] + " is not found in current platform\n");
+ System.out.println("\nWARNING: " + moduleId + " for " + archList[k] + " was not found in current platform FPD file!\n");
continue;
} else if (GlobalData.isModuleBuilt(fpdModuleId)) {
return;
@@ -265,7 +265,7 @@ public class GenBuildTask extends Ant {
// don't do anything if no tools found
//
if (GlobalData.isCommandSet(targetList[i], toolchainList[j], archList[k]) == false) {
- System.out.println("Warning: No build issued. No tools found for [target=" + targetList[i] + " toolchain=" + toolchainList[j] + " arch=" + archList[k] + "]\n");
+ System.out.println("Warning: No build issued. No tools were found for [target=" + targetList[i] + " toolchain=" + toolchainList[j] + " arch=" + archList[k] + "]\n");
continue;
}
@@ -336,7 +336,7 @@ public class GenBuildTask extends Ant {
String filename = getProject().getProperty("PLATFORM_FILE");
if (filename == null){
- throw new BuildException("Plese set ACTIVE_PLATFORM if you want to build a single module. ");
+ throw new BuildException("Please set ACTIVE_PLATFORM in the file: Tools/Conf/target.txt if you want to build a single module!");
}
PlatformIdentification platformId = GlobalData.getPlatform(filename);
@@ -608,7 +608,7 @@ public class GenBuildTask extends Ant {
// then call the exist BaseName_build.xml directly.
//
if (moduleId.getModuleType().equalsIgnoreCase("USER_DEFINED")) {
- GlobalData.log.info("Call user-defined " + moduleId.getName() + "_build.xml");
+ GlobalData.log.info("Calling user-defined " + moduleId.getName() + "_build.xml");
Ant ant = new Ant();
ant.setProject(getProject());
ant.setAntfile(getProject().getProperty("MODULE_DIR") + File.separatorChar + moduleId.getName() + "_build.xml");
@@ -638,7 +638,7 @@ public class GenBuildTask extends Ant {
// then call the exist BaseName_build.xml directly.
//
if (moduleId.getModuleType().equalsIgnoreCase("USER_DEFINED")) {
- GlobalData.log.info("Call user-defined " + moduleId.getName() + "_build.xml");
+ GlobalData.log.info("Calling user-defined " + moduleId.getName() + "_build.xml");
Ant ant = new Ant();
ant.setProject(getProject());
ant.setAntfile(getProject().getProperty("MODULE_DIR") + File.separatorChar + moduleId.getName() + "_build.xml");
diff --git a/Tools/Source/GenBuild/org/tianocore/build/ModuleBuildFileGenerator.java b/Tools/Source/GenBuild/org/tianocore/build/ModuleBuildFileGenerator.java
index cdd8814..df1af18 100644
--- a/Tools/Source/GenBuild/org/tianocore/build/ModuleBuildFileGenerator.java
+++ b/Tools/Source/GenBuild/org/tianocore/build/ModuleBuildFileGenerator.java
@@ -54,10 +54,10 @@ public class ModuleBuildFileGenerator {
/// The information at the header of <em>build.xml</em>.
///
private String info = "DO NOT EDIT \n"
- + "File auto-generated by build utility\n"
+ + "This file is auto-generated by the build utility\n"
+ "\n"
+ "Abstract:\n"
- + "Auto-generated ANT build file for building of EFI Modules/Platforms\n";
+ + "Auto-generated ANT build file for build EFI Modules and Platforms\n";
private FpdModuleIdentification fpdModuleId;
@@ -224,7 +224,7 @@ public class ModuleBuildFileGenerator {
xformer.setOutputProperty(OutputKeys.INDENT, "yes");
xformer.transform(source, result);
} catch (Exception ex) {
- throw new BuildException("Module [" + fpdModuleId.getModule().getName() + "] generating build file failed.\n" + ex.getMessage());
+ throw new BuildException("Generating the module [" + fpdModuleId.getModule().getName() + "] build.xml file failed!.\n" + ex.getMessage());
}
}
diff --git a/Tools/Source/GenBuild/org/tianocore/build/autogen/AutoGen.java b/Tools/Source/GenBuild/org/tianocore/build/autogen/AutoGen.java
index 2d7bb09..0704bf4 100644
--- a/Tools/Source/GenBuild/org/tianocore/build/autogen/AutoGen.java
+++ b/Tools/Source/GenBuild/org/tianocore/build/autogen/AutoGen.java
@@ -243,7 +243,7 @@ public class AutoGen {
libGenAutogenH();
} catch (Exception e) {
throw new BuildException(
- "Faile to create library AutoGen.c & AutoGen.h!\n"
+ "Failed to create library AutoGen.c & AutoGen.h!\n"
+ e.getMessage());
}
}
@@ -672,7 +672,7 @@ public class AutoGen {
libClassList[i]);
if (includerName == null) {
throw new AutoGenException("Can not find library class ["
- + libClassList[i] + "] declaration in every packages. ");
+ + libClassList[i] + "] declaration in any SPD package. ");
}
for (int j = 0; j < includerName.length; j++) {
String includeNameStr = includerName[j];
@@ -714,7 +714,7 @@ public class AutoGen {
if (pkgHeader == null) {
throw new AutoGenException("Can not find package ["
+ packageNameList[i]
- + "] declaration in every packages. ");
+ + "] declaration in any SPD package. ");
} else if (!pkgHeader.equalsIgnoreCase("")) {
includeStr = CommonDefinition.include + " <" + pkgHeader
+ ">\r\n";
@@ -751,7 +751,7 @@ public class AutoGen {
case CommonDefinition.ModuleTypePeiCore:
if (entryPointList == null ||entryPointList.length != 1 ) {
throw new BuildException(
- "Module type = 'PEI_CORE', only have one module entry point!");
+ "Module type = 'PEI_CORE', can have only one module entry point!");
} else {
fileBuffer.append("EFI_STATUS\r\n");
fileBuffer.append("EFIAPI\r\n");
@@ -783,7 +783,7 @@ public class AutoGen {
fileBuffer.append("const UINT32 _gUefiDriverRevision = 0;\r\n");
if (entryPointList == null || entryPointList.length != 1) {
throw new BuildException(
- "Module type = 'DXE_CORE', only have one module entry point!");
+ "Module type = 'DXE_CORE', can have only one module entry point!");
} else {
fileBuffer.append("VOID\r\n");
@@ -1231,7 +1231,7 @@ public class AutoGen {
// If can't find Ppi GUID declaration in every package
//
throw new AutoGenException("Can not find Ppi GUID ["
- + ppiKeyWord + "] declaration in every packages. ");
+ + ppiKeyWord + "] declaration in any SPD package!");
}
}
}
@@ -1288,7 +1288,7 @@ public class AutoGen {
// If can't find protocol GUID declaration in every package
//
throw new BuildException("Can not find protocol Guid ["
- + protocolKeyWord + "] declaration in every packages. ");
+ + protocolKeyWord + "] declaration in any SPD package!");
}
}
}
@@ -1331,7 +1331,7 @@ public class AutoGen {
// If can't find GUID declaration in every package
//
throw new AutoGenException("Can not find Guid [" + guidKeyWord
- + "] declaration in every packages. ");
+ + "] declaration in any SPD package. ");
}
}
@@ -1499,7 +1499,7 @@ public class AutoGen {
fileBuffer.append(" (ImageHandle, SystemTable);\r\n");
break;
default:
- EdkLog.log(EdkLog.EDK_INFO,"Autogen don't know how to deal with module type -"+ moduleType + " !");
+ EdkLog.log(EdkLog.EDK_INFO,"Autogen doesn't know how to deal with module type - " + moduleType + "!");
}
fileBuffer.append(" ASSERT_EFI_ERROR (Status);\r\n");
}
@@ -1985,7 +1985,7 @@ public class AutoGen {
fis.close();
fos.close();
}else {
- throw new AutoGenException("The flashMap.h file don't exist!!");
+ throw new AutoGenException("The file, flashMap.h doesn't exist!");
}
} catch (Exception e){
throw new AutoGenException(e.getMessage());
diff --git a/Tools/Source/GenBuild/org/tianocore/build/autogen/AutogenLibOrder.java b/Tools/Source/GenBuild/org/tianocore/build/autogen/AutogenLibOrder.java
index 5ab585c..8aad4cc 100644
--- a/Tools/Source/GenBuild/org/tianocore/build/autogen/AutogenLibOrder.java
+++ b/Tools/Source/GenBuild/org/tianocore/build/autogen/AutogenLibOrder.java
@@ -85,7 +85,7 @@ public class AutogenLibOrder {
if (this.libInstanceMap.containsKey(libraryList[i])) {
throw new Exception(
libraryList[i].getName()
- + "this library instance is already exist, please check you library instance list!");
+ + "this library instance already exists, please check the library instance list!");
} else {
this.libInstanceMap.put(libraryList[i], classStr);
}
@@ -102,8 +102,8 @@ public class AutogenLibOrder {
System.out.println(libClassDeclList[j]
+ " class is already implement by "
+ this.libClassMap.get(libClassDeclList[j]));
- throw new Exception(libClassDeclList
- + " is already have library instance!");
+ throw new Exception("Library Class: " + libClassDeclList
+ + " already has a library instance!");
} else {
this.libClassMap.put(libClassDeclList[j], libraryList[i]);
}
diff --git a/Tools/Source/GenBuild/org/tianocore/build/autogen/CommonDefinition.java b/Tools/Source/GenBuild/org/tianocore/build/autogen/CommonDefinition.java
index c6bf678..fe3b460 100644
--- a/Tools/Source/GenBuild/org/tianocore/build/autogen/CommonDefinition.java
+++ b/Tools/Source/GenBuild/org/tianocore/build/autogen/CommonDefinition.java
@@ -317,7 +317,7 @@ public class CommonDefinition {
return guid;
} else {
System.out
- .println("Check GUID Value, it don't conform to the schema!!!");
+ .println("Check GUID Value, It doesn't conform to the registry format specified in the schema!!!");
return "0";
}
@@ -351,4 +351,4 @@ public class CommonDefinition {
return desList;
}
-} \ No newline at end of file
+}
diff --git a/Tools/Source/GenBuild/org/tianocore/build/fpd/FpdParserTask.java b/Tools/Source/GenBuild/org/tianocore/build/fpd/FpdParserTask.java
index 23be330..1aaf660 100644
--- a/Tools/Source/GenBuild/org/tianocore/build/fpd/FpdParserTask.java
+++ b/Tools/Source/GenBuild/org/tianocore/build/fpd/FpdParserTask.java
@@ -130,7 +130,7 @@ public class FpdParserTask extends Task {
// Remove !!
if ( fpdFile == null) {
if (platformName == null) {
- throw new BuildException("FpdParserTask parameter error. Please specify platform name or FPD file. ");
+ throw new BuildException("FpdParserTask parameter error. Please specify either the platform name or FPD file!");
}
platformId = GlobalData.getPlatformByName(platformName);
fpdFile = platformId.getFpdFile();
@@ -300,7 +300,7 @@ public class FpdParserTask extends Task {
bw.close();
fw.close();
} catch (Exception e) {
- throw new BuildException("Generate FV file [" + fvFile.getPath() + "] failed. \n" + e.getMessage());
+ throw new BuildException("Generation of the FV file [" + fvFile.getPath() + "] failed!\n" + e.getMessage());
}
}
}
@@ -327,7 +327,7 @@ public class FpdParserTask extends Task {
XmlObject doc = XmlObject.Factory.parse(fpdFile);
if (!doc.validate()) {
- throw new BuildException("Platform Surface Area file [" + fpdFile.getPath() + "] is invalid.");
+ throw new BuildException("Platform Surface Area file [" + fpdFile.getPath() + "] format is invalid!");
}
Map<String, XmlObject> map = new HashMap<String, XmlObject>();
@@ -378,7 +378,7 @@ public class FpdParserTask extends Task {
PlatformPcdPreprocessActionForBuilding ca = new PlatformPcdPreprocessActionForBuilding();
ca.perform(platformId.getFpdFile().getPath(), ActionMessage.NULL_MESSAGE_LEVEL);
} catch (Exception e) {
- throw new BuildException("Load FPD file [" + fpdFile.getPath() + "] error. \n" + e.getMessage());
+ throw new BuildException("Parsing of the FPD file [" + fpdFile.getPath() + "] failed!\n" + e.getMessage());
}
}
diff --git a/Tools/Source/GenBuild/org/tianocore/build/fpd/PlatformBuildFileGenerator.java b/Tools/Source/GenBuild/org/tianocore/build/fpd/PlatformBuildFileGenerator.java
index b7d2881..e6a1002 100644
--- a/Tools/Source/GenBuild/org/tianocore/build/fpd/PlatformBuildFileGenerator.java
+++ b/Tools/Source/GenBuild/org/tianocore/build/fpd/PlatformBuildFileGenerator.java
@@ -53,10 +53,10 @@ public class PlatformBuildFileGenerator {
private Project project;
private String info = "DO NOT EDIT \n"
- + "File auto-generated by build utility\n"
+ + "This file is auto-generated by the build utility\n"
+ "\n"
+ "Abstract:\n"
- + "Auto-generated ANT build file for building of EFI Modules/Platforms\n";
+ + "Auto-generated ANT build file for building EFI Modules and Platforms\n";
public PlatformBuildFileGenerator(Project project, Map<FpdModuleIdentification, String> outfiles, boolean isUnified){
this.project = project;
@@ -164,7 +164,7 @@ public class PlatformBuildFileGenerator {
xformer.setOutputProperty(OutputKeys.INDENT, "yes");
xformer.transform(source, result);
} catch (Exception ex) {
- throw new BuildException("Generate " + platformName + "_build.xml failed. \n" + ex.getMessage());
+ throw new BuildException("Generation of the " + platformName + "_build.xml failed!\n" + ex.getMessage());
}
}
@@ -351,7 +351,7 @@ public class PlatformBuildFileGenerator {
//
// Deep Clean Target
//
- root.appendChild(document.createComment("Clean All target"));
+ root.appendChild(document.createComment("Target: cleanall"));
Element ele = document.createElement("target");
ele.setAttribute("name", "cleanall");
@@ -436,7 +436,7 @@ public class PlatformBuildFileGenerator {
//
// User Extensions
//
- root.appendChild(document.createComment("Pre Build Processing"));
+ root.appendChild(document.createComment("Pre-Build Processing"));
Element ele = document.createElement("target");
ele.setAttribute("name", "prebuild");
@@ -480,7 +480,7 @@ public class PlatformBuildFileGenerator {
//
// User Extensions
//
- root.appendChild(document.createComment("Post Build Processing"));
+ root.appendChild(document.createComment("Post-Build Processing"));
Element ele = document.createElement("target");
ele.setAttribute("name", "postbuild");
diff --git a/Tools/Source/GenBuild/org/tianocore/build/global/GlobalData.java b/Tools/Source/GenBuild/org/tianocore/build/global/GlobalData.java
index d83114a..1e75687 100644
--- a/Tools/Source/GenBuild/org/tianocore/build/global/GlobalData.java
+++ b/Tools/Source/GenBuild/org/tianocore/build/global/GlobalData.java
@@ -162,7 +162,7 @@ public class GlobalData {
// CONF dir + tools definition file name
//
File toolsDefFile = new File(workspaceDir + File.separatorChar + toolsDefFilename);
- System.out.println("Using file [" + toolsDefFile.getPath() + "] as tools definition file. ");
+ System.out.println("Using tool definiton file [" + toolsDefFile.getPath() + "].");
toolsDef = new ToolChainConfig(toolsDefFile);
//
@@ -175,7 +175,7 @@ public class GlobalData {
// validate FrameworkDatabaseFile
//
if (! db.validate()) {
- throw new BuildException("Framework Database file [" + dbFile.getPath() + "] is invalid.");
+ throw new BuildException("Framework Database file [" + dbFile.getPath() + "] format is invalid!");
}
//
// Get package list
@@ -208,7 +208,7 @@ public class GlobalData {
// Verify FPD file, if is invalid, throw Exception
//
if (! fpdDoc.validate()) {
- throw new BuildException("Framework Platform Surface Area file [" + fpdFile.getPath() + "] is invalid. ");
+ throw new BuildException("Framework Platform Surface Area file [" + fpdFile.getPath() + "] format is invalid!");
}
//
// We can change Map to XmlObject
@@ -225,7 +225,7 @@ public class GlobalData {
}
}
} catch (Exception e) {
- throw new BuildException("Parse workspace Database [" + dbFile.getPath() + "] Error.\n" + e.getMessage());
+ throw new BuildException("Parse WORKSPACE Database file [" + dbFile.getPath() + "] Error.\n" + e.getMessage());
}
}
@@ -257,7 +257,7 @@ public class GlobalData {
}
}
if (msaFile == null){
- throw new BuildException("Can't find Module [" + moduleId.getName() + "] in all packages. ");
+ throw new BuildException("Can't find Module [" + moduleId.getName() + "] in any SPD package!");
}
else {
return msaFile;
@@ -283,7 +283,7 @@ public class GlobalData {
}
}
if (packageId == null){
- throw new BuildException("Can't find Module [" + moduleId.getName() + "] in all packages. ");
+ throw new BuildException("Can't find Module [" + moduleId.getName() + "] in any SPD package!");
}
else {
return packageId;
@@ -387,7 +387,7 @@ public class GlobalData {
public synchronized static Map<String, XmlObject> getNativeMsa(File msaFile) throws BuildException {
if (! msaFile.exists()) {
- throw new BuildException("Surface Area file [" + msaFile.getPath() + "] can't found.");
+ throw new BuildException("Module Surface Area file [" + msaFile.getPath() + "] can't be found!");
}
try {
ModuleSurfaceAreaDocument doc = (ModuleSurfaceAreaDocument)XmlObject.Factory.parse(msaFile);
@@ -395,7 +395,7 @@ public class GlobalData {
// Validate File if they accord with XML Schema
//
if ( ! doc.validate()){
- throw new BuildException("Module Surface Area file [" + msaFile.getPath() + "] is invalid.");
+ throw new BuildException("Module Surface Area file [" + msaFile.getPath() + "] format is invalid!");
}
//
// parse MSA file
@@ -475,7 +475,7 @@ public class GlobalData {
// If can't find library class declaration in every package
//
throw new BuildException("Can not find library class [" + name
- + "] declaration in every packages. ");
+ + "] declaration in any SPD package!");
}
/**
@@ -581,7 +581,7 @@ public class GlobalData {
return platformId;
}
}
- throw new BuildException("Can't find platform [" + name + "] in current workspace database. ");
+ throw new BuildException("Can't find platform [" + name + "] in the current WORKSPACE database!");
}
public synchronized static PlatformIdentification getPlatform(String filename) throws BuildException {
@@ -593,7 +593,7 @@ public class GlobalData {
return platformId;
}
}
- throw new BuildException("Can't find platform file [" + filename + "] in current workspace database. ");
+ throw new BuildException("Can't find platform file [" + filename + "] in the current WORKSPACE database!");
}
public synchronized static PackageIdentification refreshPackageIdentification(PackageIdentification packageId) throws BuildException {
@@ -606,7 +606,7 @@ public class GlobalData {
return packageId;
}
}
- throw new BuildException("Can't find package GUID value " + packageId.getGuid() + " under current workspace. ");
+ throw new BuildException("Can't find package GUID value " + packageId.getGuid() + " in the current workspace!");
}
public synchronized static ModuleIdentification refreshModuleIdentification(ModuleIdentification moduleId) throws BuildException {
@@ -617,7 +617,7 @@ public class GlobalData {
moduleId.setPackage(packageId);
Spd spd = spdTable.get(packageId);
if (spd == null) {
- throw new BuildException("Can't find package GUID value " + packageId.getGuid() + " under current workspace. ");
+ throw new BuildException("Can't find package GUID value " + packageId.getGuid() + " in the current workspace!");
}
Set<ModuleIdentification> modules = spd.getModules();
Iterator<ModuleIdentification> iter = modules.iterator();
@@ -630,7 +630,7 @@ public class GlobalData {
return moduleId;
}
}
- throw new BuildException("Can't find module GUID value " + moduleId.getGuid() + " in " + packageId + " under current workspace. ");
+ throw new BuildException("Can't find module GUID value " + moduleId.getGuid() + " in package, " + packageId + ", in the current workspace!");
}
public synchronized static Set<PackageIdentification> getPackageList(){
diff --git a/Tools/Source/GenBuild/org/tianocore/build/global/OnDependency.java b/Tools/Source/GenBuild/org/tianocore/build/global/OnDependency.java
index ba388a1..4177b50 100644
--- a/Tools/Source/GenBuild/org/tianocore/build/global/OnDependency.java
+++ b/Tools/Source/GenBuild/org/tianocore/build/global/OnDependency.java
@@ -90,7 +90,7 @@ public class OnDependency extends Task {
} else {
File srcFile = new File(srcFileName);
if (!srcFile.exists()) {
- throw new BuildException(srcFileName + " doesn't exist !!!");
+ throw new BuildException("Source File name: " + srcFileName + " doesn't exist!!!");
}
srcTimeStamp = srcFile.lastModified();
timeStampCache.put(srcFileName, new Long(srcTimeStamp));
diff --git a/Tools/Source/GenBuild/org/tianocore/build/global/Spd.java b/Tools/Source/GenBuild/org/tianocore/build/global/Spd.java
index 3e09f58..8772385 100644
--- a/Tools/Source/GenBuild/org/tianocore/build/global/Spd.java
+++ b/Tools/Source/GenBuild/org/tianocore/build/global/Spd.java
@@ -93,7 +93,7 @@ public class Spd {
// If specified package file not exists
//
if ( ! packageFile.exists()) {
- throw new BuildException("Package file [" + packageFile.getPath() + "] not exists. ");
+ throw new BuildException("Package file [" + packageFile.getPath() + "] does not exist!");
}
try {
XmlObject spdDoc = XmlObject.Factory.parse(packageFile);
@@ -101,7 +101,7 @@ public class Spd {
// Verify SPD file, if is invalid, throw Exception
//
if (! spdDoc.validate()) {
- throw new BuildException("Package Surface Area file [" + packageFile.getPath() + "] is invalid. ");
+ throw new BuildException("Package Surface Area file [" + packageFile.getPath() + "] format is invalid!");
}
// We can change Map to XmlObject
Map<String, XmlObject> spdDocMap = new HashMap<String, XmlObject>();
@@ -127,7 +127,7 @@ public class Spd {
moduleId.setPackage(packageId);
moduleId.setMsaFile(msaFile);
if (msaInfo.containsKey(moduleId)) {
- throw new BuildException("Find two modules with the same GUID and Version in " + packageId + ". They are [" + msaInfo.get(moduleId) + "] and [" + msaFile + "] ");
+ throw new BuildException("Found two modules with the same GUID and Version in package " + packageId + ".\nThey are module [" + msaInfo.get(moduleId) + "] and MSA file [" + msaFile + "]!");
}
msaInfo.put(moduleId, msaFile);
}
@@ -199,7 +199,7 @@ public class Spd {
}
catch (Exception e) {
e.setStackTrace(e.getStackTrace());
- throw new BuildException("Parse package description file [" + packageId.getSpdFile() + "] Error.\n"
+ throw new BuildException("Parse of the package description file [" + packageId.getSpdFile() + "] failed!\n"
+ e.getMessage());
}
}
diff --git a/Tools/Source/GenBuild/org/tianocore/build/global/VariableTask.java b/Tools/Source/GenBuild/org/tianocore/build/global/VariableTask.java
index f830e0a..021b8d5 100644
--- a/Tools/Source/GenBuild/org/tianocore/build/global/VariableTask.java
+++ b/Tools/Source/GenBuild/org/tianocore/build/global/VariableTask.java
@@ -63,7 +63,7 @@ public class VariableTask extends Task {
*/
public void execute() throws BuildException {
if (name == null || value == null) {
- throw new BuildException("Name or value must not null.");
+ throw new BuildException("Name or value cannot be null.");
}
getProject().setProperty(name, value);
}
diff --git a/Tools/Source/GenBuild/org/tianocore/build/pcd/action/PCDAutoGenAction.java b/Tools/Source/GenBuild/org/tianocore/build/pcd/action/PCDAutoGenAction.java
index 5928061..5c7b29a 100644
--- a/Tools/Source/GenBuild/org/tianocore/build/pcd/action/PCDAutoGenAction.java
+++ b/Tools/Source/GenBuild/org/tianocore/build/pcd/action/PCDAutoGenAction.java
@@ -171,7 +171,7 @@ public class PCDAutoGenAction extends BuildAction {
}
ActionMessage.debug(this,
- "PCD memory database contains " + dbManager.getDBSize() + " PCD tokens");
+ "PCD memory database contains " + dbManager.getDBSize() + " PCD tokens.");
generateAutogenForModule();
}
@@ -228,9 +228,9 @@ public class PCDAutoGenAction extends BuildAction {
// All library's PCD should instanted in module's <ModuleSA> who
// use this library instance. If not, give errors.
//
- throw new BuildActionException (String.format("Module %s use library instance %s, the PCD %s " +
- "is required by this library instance, but can not find " +
- "it in the %s's <ModuleSA> in FPD file!",
+ throw new BuildActionException (String.format("Module %s using library instance %s; the PCD %s " +
+ "is required by this library instance, but can not be found " +
+ "in the %s's <ModuleSA> in the FPD file!",
MemoryDatabaseManager.CurrentModuleName,
moduleName,
pcdNameArrayInMsa[index],
diff --git a/Tools/Source/GenBuild/org/tianocore/build/pcd/action/PcdDatabase.java b/Tools/Source/GenBuild/org/tianocore/build/pcd/action/PcdDatabase.java
index a42ef92..0460efd 100644
--- a/Tools/Source/GenBuild/org/tianocore/build/pcd/action/PcdDatabase.java
+++ b/Tools/Source/GenBuild/org/tianocore/build/pcd/action/PcdDatabase.java
@@ -1326,7 +1326,7 @@ public class PcdDatabase {
case POINTER:
return "UINT8";
default:
- throw new EntityException("Unknown type in getDataTypeCDeclaration");
+ throw new EntityException("Unknown DatumType in getDataTypeCDeclaration");
}
}
@@ -1486,7 +1486,7 @@ public class PcdDatabase {
retStr = retStr +"\r\n" + str;
}
} catch (Exception ex) {
- throw new EntityException("Fatal error when generating PcdDatabase Common Definitions");
+ throw new EntityException("Fatal error when generating PcdDatabase Common Definitions!");
}
return retStr;
@@ -1509,7 +1509,7 @@ public class PcdDatabase {
retStr = retStr +"\r\n" + str;
}
} catch (Exception ex) {
- throw new EntityException("Fatal error when generating PcdDatabase Dxe Definitions");
+ throw new EntityException("Fatal error when generating PcdDatabase Dxe Definitions!");
}
return retStr;
@@ -1532,7 +1532,7 @@ public class PcdDatabase {
retStr = retStr +"\r\n" + str;
}
} catch (Exception ex) {
- throw new EntityException("Fatal error when generating PcdDatabase Pei Definitions");
+ throw new EntityException("Fatal error when generating PcdDatabase Pei Definitions!");
}
return retStr;
@@ -1585,7 +1585,7 @@ public class PcdDatabase {
if ((uuidString.charAt(0) == '0') && ((uuidString.charAt(1) == 'x') || (uuidString.charAt(1) == 'X'))) {
splitStringArray = uuidString.split("," );
if (splitStringArray.length != 11) {
- throw new EntityException ("[FPD file error] Wrong format for UUID string: " + uuidString);
+ throw new EntityException ("[FPD file error] Wrong format for GUID string: " + uuidString);
}
//
@@ -1626,4 +1626,4 @@ public class PcdDatabase {
return UUID.fromString(uuidString);
}
-} \ No newline at end of file
+}
diff --git a/Tools/Source/GenBuild/org/tianocore/build/pcd/action/PlatformPcdPreprocessActionForBuilding.java b/Tools/Source/GenBuild/org/tianocore/build/pcd/action/PlatformPcdPreprocessActionForBuilding.java
index 0b2fd0b..a0a40eb 100644
--- a/Tools/Source/GenBuild/org/tianocore/build/pcd/action/PlatformPcdPreprocessActionForBuilding.java
+++ b/Tools/Source/GenBuild/org/tianocore/build/pcd/action/PlatformPcdPreprocessActionForBuilding.java
@@ -121,7 +121,7 @@ public class PlatformPcdPreprocessActionForBuilding extends PlatformPcdPreproces
**/
public void execute() throws PlatformPcdPreprocessBuildException {
- String errorMessageHeader = "Fail to initialize Pcd memory database for building. Because:";
+ String errorMessageHeader = "Failed to initialize the Pcd memory database because: ";
String errorsForPreprocess = null;
//
@@ -170,7 +170,7 @@ public class PlatformPcdPreprocessActionForBuilding extends PlatformPcdPreproces
try {
tokenSpaceStrRet = GlobalData.getGuidInfoFromCname(guidCName);
} catch ( Exception e ) {
- throw new PlatformPcdPreprocessException ("Failed get Guid CName " + guidCName + "from SPD file!");
+ throw new PlatformPcdPreprocessException ("Failed to get Guid CName " + guidCName + " from the SPD file!");
}
return tokenSpaceStrRet;
}
@@ -303,8 +303,8 @@ public class PlatformPcdPreprocessActionForBuilding extends PlatformPcdPreproces
dynamicPcdBuildDefinitions = fpdDocInstance.getPlatformSurfaceArea().getDynamicPcdBuildDefinitions();
if (dynamicPcdBuildDefinitions == null) {
- exceptionString = String.format("[FPD file error] There are no <PcdDynamicBuildDescriptions> in FPD file but contains Dynamic type "+
- "PCD entry %s in module %s!",
+ exceptionString = String.format("[FPD file error] There are no <PcdDynamicBuildDescriptions> elements in FPD file but there are Dynamic type "+
+ "PCD entries %s in module %s!",
token.cName,
moduleName);
putError(exceptionString);
@@ -374,11 +374,11 @@ public class PlatformPcdPreprocessActionForBuilding extends PlatformPcdPreproces
File file = null;
if (fpdFilePath == null) {
- throw new PlatformPcdPreprocessBuildException("WorkspacePath and FPDFileName should be blank for CollectPCDAtion!");
+ throw new PlatformPcdPreprocessBuildException("WorkspacePath and FPDFileName should be empty for CollectPCDAtion!");
}
if (fpdFilePath.length() == 0) {
- throw new PlatformPcdPreprocessBuildException("WorkspacePath and FPDFileName should be blank for CollectPCDAtion!");
+ throw new PlatformPcdPreprocessBuildException("WorkspacePath and FPDFileName should be empty for CollectPCDAtion!");
}
file = new File(fpdFilePath);
diff --git a/Tools/Source/GenBuild/org/tianocore/build/toolchain/ConfigReader.java b/Tools/Source/GenBuild/org/tianocore/build/toolchain/ConfigReader.java
index 0097890..66d38f9 100644
--- a/Tools/Source/GenBuild/org/tianocore/build/toolchain/ConfigReader.java
+++ b/Tools/Source/GenBuild/org/tianocore/build/toolchain/ConfigReader.java
@@ -89,7 +89,7 @@ public class ConfigReader {
valueList.add(str.substring(index + 1).trim());
}
} catch (Exception e) {
- throw new EdkException("Process file [" + filename + "] error. \n" + e.getMessage());
+ throw new EdkException("ERROR Processing file [" + filename + "].\n" + e.getMessage());
}
String[][] definitions = new String[2][keyList.size()];
@@ -126,7 +126,7 @@ public class ConfigReader {
map.put(key, value);
}
} catch (Exception e) {
- throw new EdkException("Process file [" + ConfigFile.getAbsolutePath() + "] error. \n" + e.getMessage());
+ throw new EdkException("ERROR Processing file [" + ConfigFile.getAbsolutePath() + "].\n" + e.getMessage());
}
return map;