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.

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

Workaround for 'OSError: [Errno 18] Invalid cross-device link:' (#806) #1130

Merged
merged 4 commits into from
Apr 29, 2020

Conversation

iromeo
Copy link
Contributor

@iromeo iromeo commented Mar 19, 2020

See detailed description at #806 (comment)

shutil.copy(..) leads to OSError: [Errno 18] Invalid cross-device link when src and dst are in different filesystems, is reproducible in python 3.6 and 3.8. Workaround is to split it in shutil.copy() and os.remove()

Many thanks to contributing to MultiQC!

Please fill in the appropriate checklist below (delete whatever is not relevant). These are the most common things I request on pull requests (PRs).

If this PR is not a new module

`shutil.copy(..)` leads to `OSError: [Errno 18] Invalid cross-device link` when src and dst are in different filesystems, is reproducible in python 3.6 and 3.8. Workaround is to split it in `shutil.copy()` and `os.remove()`
@iromeo iromeo changed the title fix: workaround for https://github.com/ewels/MultiQC/issues/806 Workaround for 'OSError: [Errno 18] Invalid cross-device link:' (#806) Mar 19, 2020
multiqc/multiqc.py Outdated Show resolved Hide resolved
Replaced tmpdir copy behaviour to work properly with directories and contents

Fixed up directory naming behaviour for multiqc_plots to work consistently with multiqc_data and report
@ewels
Copy link
Member

ewels commented Apr 29, 2020

Thanks @iromeo!

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.

2 participants