Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: instead of moving mill cp and then remove #422

Merged
merged 1 commit into from
Dec 2, 2022
Merged

fix: instead of moving mill cp and then remove #422

merged 1 commit into from
Dec 2, 2022

Commits on Dec 2, 2022

  1. fix: instead of moving mill cp and then remove 

    It looks like there are situations when dealing with mounted drives
you get the following error:

```
Error: EXDEV: cross-device link not permitted, rename '/runner/_work/_temp/9b722d14-6935-45f9-ab1a-526911088cbe' -> '/home/runner/bin/mill'
```

From googling around it looks like we can avoid this by instead
moving the file first and then removing it.

For reference

- https://stackoverflow.com/questions/44146393/error-exdev-cross-device-link-not-permitted-rename-nodejs
- shelljs/shelljs#191

Fixes: scala-steward-org#421
    @ckipp01
    ckipp01 committed Dec 2, 2022
    Configuration menu
    Copy the full SHA
    af50c17 View commit details
    Browse the repository at this point in the history