[EdgeTPU] Update Toolchain run notification and log (Samsung#1698)

Modify notiTitle in JobConfig.ts from 'Running onecc...' to 'Running tools...' Use active toolchain name in notification and log of ToolchainProvider _run method ONE-vscode-DCO-1.0-Signed-off-by: profornnan <[email protected]>
SSDC-onE-FiVE · Oct 11, 2023 · 0107bfb · 0107bfb
1 parent 748b4fe
commit 0107bfb
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 12 deletions.
diff --git a/src/Toolchain/JobConfig.ts b/src/Toolchain/JobConfig.ts
@@ -27,7 +27,7 @@ class JobConfig extends JobCommand {
     super(cmd);
     this.jobType = JobType.tConfig;
     this.name = "config";
-    this.notiTitle = "Running onecc...";
+    this.notiTitle = "Running tools...";
     this.valid = true;
     this.isCancelable = true;
   }

diff --git a/src/Toolchain/ToolchainEnv.ts b/src/Toolchain/ToolchainEnv.ts
@@ -181,6 +181,7 @@ class ToolchainEnv extends Env {
     return new Promise<boolean>((resolve, reject) => {
       const jobs: Array<Job> = [];
       const job = new JobConfig(toolchain.run(cfg));
+      job.notiTitle = `Running ${toolchain.info.name}`;
       job.workDir = path.dirname(cfg);
       job.successCallback = () => resolve(true);
       job.failureCallback = () => reject();

diff --git a/src/Toolchain/ToolchainProvider.ts b/src/Toolchain/ToolchainProvider.ts
@@ -316,27 +316,29 @@ export class ToolchainProvider implements vscode.TreeDataProvider<BaseNode> {
   }
 
   public _run(cfg: string): boolean {
+    const [activeToolchainEnv, activeToolchain] =
+      this.checkAvailableToolchain();
+    if (!activeToolchainEnv || !activeToolchain) {
+      return false;
+    }
+
+    const activeToolchainName = activeToolchain.info.name;
+
     /* istanbul ignore next */
     const notifySuccess = () => {
-      vscode.window.showInformationMessage("Onecc has run successfully.");
+      vscode.window.showInformationMessage(
+        `${activeToolchainName} has run successfully.`
+      );
     };
 
     /* istanbul ignore next */
     const notifyError = () => {
-      this.error("Running onecc has failed.");
+      this.error(`Running ${activeToolchainName} has failed.`);
     };
 
-    const [activeToolchainEnv, activeToolchain] =
-      this.checkAvailableToolchain();
-    if (activeToolchainEnv === undefined || activeToolchain === undefined) {
-      return false;
-    }
-
     Logger.info(
       this.tag,
-      `Run onecc with ${cfg} cfg and ${
-        activeToolchain.info.name
-      }-${activeToolchain.info.version?.str()} toolchain.`
+      `Run ${activeToolchainName}-${activeToolchain.info.version?.str()} toolchain with ${cfg} cfg.`
     );
     activeToolchainEnv.run(cfg, activeToolchain).then(
       () => notifySuccess(),