feat: repeated usage in the same job

.github/workflows/test.yml

@@ -34,7 +34,7 @@ jobs:
   test:
     runs-on: ubuntu-latest
     name: test-all # PRs can require just this job to pass, regardless of matrix changes below
-    needs: [check_dist, test_matrix, test_multi]
+    needs: [check_dist, test_matrix, test_multi, test_repeated]
     steps:
       - run: echo "Test-matrix successful" # anything with zero exit code
 
@@ -81,3 +81,36 @@ jobs:
             solc --version | grep 0.8.30;
             solc-v0.8.30 --version | grep 0.8.30;
             solc-v0.8.29 --version | grep 0.8.29;
+
+  test_repeated:
+    needs: check_dist
+    name: setup-solc-again
+    runs-on: ubuntu-latest
+    steps:
+        - uses: actions/checkout@v5
+
+        - name: Initial
+          uses: ./
+          with:
+            version: 0.8.30
+            versions: '0.8.30,0.8.29'
+
+        - name: Check initial version
+          run: |
+            solc --version | grep 0.8.30;
+            solc-v0.8.30 --version | grep 0.8.30;
+            solc-v0.8.29 --version | grep 0.8.29;
+
+        - name: Re-installation
+          uses: ./
+          with:
+            version: 0.8.29
+            # Check that solc-v0.8.30 existence doesn't cause an error, and that
+            # solc-v0.8.29 isn't removed despite not being specified.
+            versions: '0.8.30'
+
+        - name: Check re-installed version
+          run: |
+            solc --version | grep 0.8.29;
+            solc-v0.8.30 --version | grep 0.8.30;
+            solc-v0.8.29 --version | grep 0.8.29;

dist/index.js

@@ -29846,8 +29846,12 @@ try {
             throw Error(`Version ${version} not found`);
         }
         const downloaded = await toolCacheExports.downloadTool(constructURL(build.path));
-        await fs.chmod(downloaded, 0o555);
-        await Promise.all(outs.map((out) => fs.copyFile(downloaded, path.join(destDir, out))));
+        await Promise.all(outs.map(async (out) => {
+            const dest = path.join(destDir, out);
+            await fs.rm(dest, { force: true }); // allow multiple invocations
+            await fs.copyFile(downloaded, dest);
+            await fs.chmod(dest, 0o555);
+        }));
         console.info(`${version} at ${outs}`);
     }
     ;

src/index.ts

@@ -42,13 +42,14 @@ try {
         }
 
         const downloaded = await tc.downloadTool(constructURL(build.path));
-        await fs.chmod(downloaded, 0o555);
 
         await Promise.all(
-            outs.map((out) => fs.copyFile(
-                downloaded,
-                path.join(destDir, out)
-            ))
+            outs.map(async (out) => {
+                const dest = path.join(destDir, out);
+                await fs.rm(dest, { force: true }); // allow multiple invocations
+                await fs.copyFile(downloaded, dest);
+                await fs.chmod(dest, 0o555);
+            })
         );
         console.info(`${version} at ${outs}`);
     };