Skip to content

fix: symlink already exists now #420

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.

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

Closed
wants to merge 1 commit into from
Closed

Conversation

henryiii
Copy link
Contributor

@henryiii henryiii commented Apr 4, 2022

Fixes #419.

@gschaffner
Copy link

Can confirm that this fix works for my group once index.js has been recompiled.

For documentation's sake: it looks like GitHub-hosted Linux runners had PyPy2 updated from 7.3.8 to 7.3.9 a few days ago (actions/runner-images@1ede808, actions/runner-images@8863b25). PyPy 7.3.9 added a new pypy2 alias for pypy [ref], so I don't think setup-nox needs to create the symlink itself anymore.

For backwards-compatability though, perhaps setup-nox should still create a symlink if PyPy2 < 7.3.9. (Or simply try-catch the creation of the symlink instead of parsing the version string.) This won't ever occur on GitHub-hosted runners, but presumably folks may be using setup-nox on self-hosted runners.

@daisylb
Copy link
Owner

daisylb commented Apr 11, 2022

Thanks so much for filing this one!

For backwards-compatability though, perhaps setup-nox should still create a symlink if PyPy2 < 7.3.9. (Or simply try-catch the creation of the symlink instead of parsing the version string.) This won't ever occur on GitHub-hosted runners, but presumably folks may be using setup-nox on self-hosted runners.

I agree with this, and I'd be more comfortable merging this if we maybe just checked to see if the symlink exists and create it if not. I'm less worried about self-hosted runners breaking, but if we can avoid it that's definitely better.

@henryiii henryiii closed this Apr 11, 2022
@henryiii
Copy link
Contributor Author

Fixed in #421! Thanks! Needs a release though (probably after pinned deps get bumped?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

setup-nox is breaking on linux runners
3 participants