Merge pull request #2087 from Azure/jcotillo/fix_summary_path

Fix summary log invalid dir
AzureSDKAutomation · Dec 6, 2021 · 4673365 · 4673365
2 parents d6b451a + 019479a
commit 4673365
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/generator/cmd/generateall.ts b/generator/cmd/generateall.ts
@@ -34,6 +34,7 @@ executeSynchronous(async () => {
     let localPath = args['local-path'];
     let summaryPath = args['summary-log-path'];
 
+    // localPath refers to the specs repo (azure-rest-api-specs)
     if (!localPath) {
         localPath = constants.specsRepoPath;
         basePaths = await cloneAndGenerateBasePaths(localPath, constants.specsRepoUri, constants.specsRepoCommitHash);
@@ -43,8 +44,9 @@ executeSynchronous(async () => {
     }
 
     if (!summaryPath) {
-        // generate default full path
-        summaryPath = path.join(constants.specsRepoPath, 'summary.log');
+        // using 'localPath' here because at this point it is guaranteed that the folder got created (when cloneAndGenerateBasePaths function is invoked)
+        // or is an existing path
+        summaryPath = path.join(localPath, 'summary.log');
     }
 
     // resolve absolute path