Ensure the tmpdir is dropped

pkgxdev · Mar 3, 2025 · 94eefeb · 94eefeb
1 parent 64f5047
commit 94eefeb
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -150,6 +150,10 @@ jobs:
         if: ${{ matrix.os == 'ubuntu-latest' }}
         # ^^ only on one platform as wasteful otherwise
 
+      - run: if [ $(find ~/.pkgx -name .tmp\* -type d | wc -l) -gt 0 ]; then
+            exit 1;
+          fi
+
       - name: generate coverage
         run: |
           cargo install rustfilt

diff --git a/crates/lib/src/install.rs b/crates/lib/src/install.rs
@@ -114,7 +114,7 @@ where
 
     // Step 5: atomically move from temp dir to installation location
     let partial_path = format!("{}/v{}", pkg.project, pkg.version.raw);
-    fs::rename(temp_dir.into_path().join(&partial_path), &dst_path)?;
+    fs::rename(temp_dir.path().join(&partial_path), &dst_path)?;
 
     let installation = Installation {
         path: dst_path,