ATOMIC-74: Improvements to build file operation failure cases

Script/AtomicEditor/ui/modal/build/BuildOutput.ts

@@ -59,7 +59,7 @@ class BuildOutput extends ModalWindow {
         return false;
     }
 
-    textOutput: string;
+    textOutput: string = "";
     buildBase: ToolCore.BuildBase;
     outputField: Atomic.UIEditField;
 

Source/ToolCore/Build/BuildAndroid.cpp

@@ -67,7 +67,7 @@ void BuildAndroid::RunADBStartActivity()
     Subprocess* subprocess = subs->Launch(adbCommand, args, buildPath_);
     if (!subprocess)
     {
-        SendBuildFailure("BuildFailed::RunStartActivity");
+        FailBuild("BuildFailed::RunStartActivity");
         return;
     }
 
@@ -110,7 +110,7 @@ void BuildAndroid::RunADBInstall()
 
     if (!subprocess)
     {
-        SendBuildFailure("BuildFailed::RunADBInstall");
+        FailBuild("BuildFailed::RunADBInstall");
         return;
     }
 
@@ -194,7 +194,7 @@ void BuildAndroid::RunADBListDevices()
 
     if (!subprocess)
     {
-        SendBuildFailure("BuildFailed::RunADBListDevices");
+        FailBuild("BuildFailed::RunADBListDevices");
         return;
     }
 
@@ -251,7 +251,7 @@ void BuildAndroid::RunAntDebug()
 
     if (!subprocess)
     {
-        SendBuildFailure("BuildFailed::RunAntDebug");
+        FailBuild("BuildFailed::RunAntDebug");
         return;
     }
 
@@ -338,7 +338,7 @@ void BuildAndroid::Build(const String& buildPath)
 
     if (!gen.Generate())
     {
-        SendBuildFailure(gen.GetErrorText());
+        FailBuild(gen.GetErrorText());
         return;
     }
 

Source/ToolCore/Build/BuildBase.cpp

@@ -12,6 +12,7 @@
 #include "../Project/Project.h"
 #include "../ToolEnvironment.h"
 
+#include "BuildSystem.h"
 #include "BuildEvents.h"
 #include "BuildBase.h"
 #include "ResourcePackager.h"
@@ -21,7 +22,8 @@ namespace ToolCore
 
 BuildBase::BuildBase(Context * context, Project* project, PlatformID platform) : Object(context),
     platformID_(platform),
-    containsMDL_(false)
+    containsMDL_(false),
+    buildFailed_(false)
 {
     if (UseResourcePackager())
         resourcePackager_ = new ResourcePackager(context, this);
@@ -38,28 +40,130 @@ BuildBase::~BuildBase()
     }
 }
 
-void BuildBase::BuildLog(const String& message)
+bool BuildBase::BuildCreateDirectory(const String& path)
+{
+    if (buildFailed_)
+    {
+        LOGERRORF("BuildBase::BuildCreateDirectory - Attempt to create directory of failed build, %s", path.CString());
+        return false;
+    }
+
+    FileSystem* fileSystem = GetSubsystem<FileSystem>();
+
+    if (fileSystem->DirExists(path))
+        return true;
+
+    bool result = fileSystem->CreateDir(path);
+
+    if (!result)
+    {
+        FailBuild(ToString("BuildBase::BuildCreateDirectory: Unable to create directory %s", path.CString()));
+        return false;
+    }
+
+    return true;
+
+}
+
+bool BuildBase::BuildCopyFile(const String& srcFileName, const String& destFileName)
+{
+    if (buildFailed_)
+    {
+        LOGERRORF("BuildBase::BuildCopyFile - Attempt to copy file of failed build, %s", srcFileName.CString());
+        return false;
+    }
+
+    FileSystem* fileSystem = GetSubsystem<FileSystem>();
+
+    bool result = fileSystem->Copy(srcFileName, destFileName);
+
+    if (!result)
+    {
+        FailBuild(ToString("BuildBase::BuildCopyFile: Unable to copy file %s -> %s", srcFileName.CString(), destFileName.CString()));
+        return false;
+    }
+
+    return true;
+}
+
+bool BuildBase::BuildRemoveDirectory(const String& path)
+{
+    if (buildFailed_)
+    {
+        LOGERRORF("BuildBase::BuildRemoveDirectory - Attempt to remove directory of failed build, %s", path.CString());
+        return false;
+    }
+
+    FileSystem* fileSystem = GetSubsystem<FileSystem>();
+    if (!fileSystem->DirExists(path))
+        return true;
+
+    bool result = fileSystem->RemoveDir(path, true);
+
+    if (!result)
+    {
+        FailBuild(ToString("BuildBase::BuildRemoveDirectory: Unable to remove directory %s", path.CString()));
+        return false;
+    }
+
+    return true;
+}
+
+void BuildBase::BuildLog(const String& message, bool sendEvent)
 {
     buildLog_.Push(message);
+
+    if (sendEvent)
+    {
+        String colorMsg = ToString("<color #D4FB79>%s</color>\n", message.CString());
+        VariantMap buildOutput;
+        buildOutput[BuildOutput::P_TEXT] = colorMsg;
+        SendEvent(E_BUILDOUTPUT, buildOutput);
+    }
+
 }
 
-void BuildBase::BuildWarn(const String& warning)
+void BuildBase::BuildWarn(const String& warning, bool sendEvent)
 {
     buildWarnings_.Push(warning);
+
+    if (sendEvent)
+    {
+        String colorMsg = ToString("<color #FFFF00>%s</color>\n", warning.CString());
+        VariantMap buildOutput;
+        buildOutput[BuildOutput::P_TEXT] = colorMsg;
+        SendEvent(E_BUILDOUTPUT, buildOutput);
+    }
+
 }
 
-void BuildBase::BuildError(const String& error)
+void BuildBase::BuildError(const String& error, bool sendEvent)
 {
     buildErrors_.Push(error);
+
+    if (sendEvent)
+    {
+        String colorMsg = ToString("<color #FF0000>%s</color>\n", error.CString());
+        VariantMap buildOutput;
+        buildOutput[BuildOutput::P_TEXT] = colorMsg;
+        SendEvent(E_BUILDOUTPUT, buildOutput);
+    }
 }
 
-void BuildBase::SendBuildFailure(const String& message)
+void BuildBase::FailBuild(const String& message)
 {
+    if (buildFailed_)
+    {
+        LOGERRORF("BuildBase::FailBuild - Attempt to fail already failed build: %s", message.CString());
+        return;
+    }
+
+    buildFailed_ = true;
+
+    BuildError(message);
 
-    VariantMap buildError;
-    buildError[BuildComplete::P_PLATFORMID] = platformID_;
-    buildError[BuildComplete::P_MESSAGE] = message;
-    SendEvent(E_BUILDCOMPLETE, buildError);
+    BuildSystem* buildSystem = GetSubsystem<BuildSystem>();
+    buildSystem->BuildComplete(platformID_, buildPath_, false, message);
 
 }
 

Source/ToolCore/Build/BuildBase.h

@@ -39,16 +39,21 @@ public:
     // will warn on name conflicts
     void AddResourceDir(const String& dir);
 
-    void BuildLog(const String& message);
-    void BuildWarn(const String& warning);
-    void BuildError(const String& error);
+    void BuildLog(const String& message, bool sendEvent = true);
+    void BuildWarn(const String& warning, bool sendEvent = true);
+    void BuildError(const String& error, bool sendEvent = true);
 
     /// Converts subprocess output event to a buildoutput event
     void HandleSubprocessOutputEvent(StringHash eventType, VariantMap& eventData);
 
 protected:
 
-    void SendBuildFailure(const String& message);
+    bool BuildRemoveDirectory(const String& path);
+    bool BuildCreateDirectory(const String& path);
+    bool BuildCopyFile(const String& srcFileName, const String& destFileName);
+
+    /// Fail the current build
+    void FailBuild(const String& message);
 
     void GenerateResourcePackage(const String& resourcePackagePath);
 
@@ -65,6 +70,7 @@ private:
 
     PlatformID platformID_;
 
+    bool buildFailed_;
     Vector<String> buildLog_;
     Vector<String> buildWarnings_;
     Vector<String> buildErrors_;

Source/ToolCore/Build/BuildIOS.cpp

@@ -160,7 +160,7 @@ void BuildIOS::RunConvertPList()
 
     if (!subprocess)
     {
-        SendBuildFailure("BuildFailed::RunConvertPList");
+        FailBuild("BuildFailed::RunConvertPList");
         return;
     }
 
@@ -210,7 +210,7 @@ void BuildIOS::RunCodeSign()
 
     if (!subprocess)
     {
-        SendBuildFailure("BuildFailed::RunCodeSign");
+        FailBuild("BuildFailed::RunCodeSign");
         return;
     }
 
@@ -259,7 +259,7 @@ void BuildIOS::RunDeploy()
 
     if (!subprocess)
     {
-        SendBuildFailure("BuildFailed::RunDeploy");
+        FailBuild("BuildFailed::RunDeploy");
         return;
     }
 

Source/ToolCore/Build/BuildMac.cpp

@@ -58,37 +58,44 @@ void BuildMac::Build(const String& buildPath)
 
     buildPath_ = AddTrailingSlash(buildPath) + GetBuildSubfolder();
 
-    VariantMap buildOutput;
-    buildOutput[BuildOutput::P_TEXT] = "\n\n<color #D4FB79>Starting Mac Deployment</color>\n\n";
-    SendEvent(E_BUILDOUTPUT, buildOutput);
+    BuildLog("Starting Mac Deployment");
 
     Initialize();
 
     BuildSystem* buildSystem = GetSubsystem<BuildSystem>();
 
+    if (!BuildRemoveDirectory(buildPath_))
+        return;
+
     FileSystem* fileSystem = GetSubsystem<FileSystem>();
-    if (fileSystem->DirExists(buildPath_))
-        fileSystem->RemoveDir(buildPath_, true);
 
     String appSrcPath = tenv->GetPlayerAppFolder();
 
-    fileSystem->CreateDir(buildPath_);
+    if (!BuildCreateDirectory(buildPath_))
+        return;
 
     buildPath_ += "/AtomicPlayer.app";
 
-    fileSystem->CreateDir(buildPath_);
-
-    fileSystem->CreateDir(buildPath_ + "/Contents");
-    fileSystem->CreateDir(buildPath_ + "/Contents/MacOS");
-    fileSystem->CreateDir(buildPath_ + "/Contents/Resources");
+    if (!BuildCreateDirectory(buildPath_))
+        return;
+    if (!BuildCreateDirectory(buildPath_ + "/Contents"))
+        return;
+    if (!BuildCreateDirectory(buildPath_ + "/Contents/MacOS"))
+        return;
+    if (!BuildCreateDirectory(buildPath_ + "/Contents/Resources"))
+        return;
 
     String resourcePackagePath = buildPath_ + "/Contents/Resources/AtomicResources" + PAK_EXTENSION;
     GenerateResourcePackage(resourcePackagePath);
 
-    fileSystem->Copy(appSrcPath + "/Contents/Resources/Atomic.icns", buildPath_ + "/Contents/Resources/Atomic.icns");
+    if (!BuildCopyFile(appSrcPath + "/Contents/Resources/Atomic.icns", buildPath_ + "/Contents/Resources/Atomic.icns"))
+        return;
+
+    if (!BuildCopyFile(appSrcPath + "/Contents/Info.plist", buildPath_ + "/Contents/Info.plist"))
+        return;
 
-    fileSystem->Copy(appSrcPath + "/Contents/Info.plist", buildPath_ + "/Contents/Info.plist");
-    fileSystem->Copy(appSrcPath + "/Contents/MacOS/AtomicPlayer", buildPath_ + "/Contents/MacOS/AtomicPlayer");
+    if (!BuildCopyFile(appSrcPath + "/Contents/MacOS/AtomicPlayer", buildPath_ + "/Contents/MacOS/AtomicPlayer"))
+        return;
 
 #ifdef ATOMIC_PLATFORM_OSX
     Vector<String> args;
@@ -97,8 +104,7 @@ void BuildMac::Build(const String& buildPath)
     fileSystem->SystemRun("chmod", args);
 #endif
 
-    buildOutput[BuildOutput::P_TEXT] = "\n\n<color #D4FB79>Mac Deployment Complete</color>\n\n";
-    SendEvent(E_BUILDOUTPUT, buildOutput);
+    BuildLog("Mac Deployment Complete");
 
     buildPath_ = buildPath + "/Mac-Build";
     buildSystem->BuildComplete(PLATFORMID_MAC, buildPath_);

Source/ToolCore/Build/ResourcePackager.cpp

@@ -55,7 +55,7 @@ ResourcePackager::~ResourcePackager()
 
 bool ResourcePackager::WritePackageFile(const String& destFilePath)
 {
-    buildBase_->BuildLog("Writing package");
+    buildBase_->BuildLog("Writing package", false);
 
     SharedPtr<File> dest(new File(context_, destFilePath, FILE_WRITE));
     if (!dest->IsOpen())
@@ -149,7 +149,7 @@ bool ResourcePackager::WritePackageFile(const String& destFilePath)
             pos += unpackedSize;
         }
 
-        buildBase_->BuildLog(entry->absolutePath_ + " in " + String(dataSize) + " out " + String(totalPackedBytes));
+        buildBase_->BuildLog(entry->absolutePath_ + " in " + String(dataSize) + " out " + String(totalPackedBytes), false);
         }
     //}
 
@@ -171,6 +171,7 @@ bool ResourcePackager::WritePackageFile(const String& destFilePath)
         dest->WriteUInt(entry->checksum_);
     }
 
+    buildBase_->BuildLog("Resource Package:");
     buildBase_->BuildLog("Number of files " + String(resourceEntries_.Size()));
     buildBase_->BuildLog("File data size " + String(totalDataSize));
     buildBase_->BuildLog("Package size " + String(dest->GetSize()));