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

OSError when downloading files (Qubes only) #83

Closed
redshiftzero opened this issue Oct 30, 2018 · 2 comments
Closed

OSError when downloading files (Qubes only) #83

redshiftzero opened this issue Oct 30, 2018 · 2 comments
Assignees
@redshiftzero
Labels
bug Something isn't working qubes needed
Milestone
0.1.0alpha

Comments

@redshiftzero
Copy link
Contributor

If one is developing in Qubes using the run.sh script, when you download files you get:

'OSError: [Errno 18] Invalid cross-device link'

Due to the fact that /tmp is on a different filesystem than /home/user, so we can't use os.rename and instead should use shutil.move
@redshiftzero redshiftzero added bug Something isn't working qubes needed labels Oct 30, 2018
@heartsucker
Copy link
Contributor

Does this error originate in the SDK or in the client code?

@redshiftzero redshiftzero self-assigned this Oct 30, 2018
@redshiftzero
Copy link
Contributor Author

it's in the client code here, i have this fixed in a branch on my workstation, will probably push up as part of #66 (it's a one line fix, just wanted to alert others developing to this bug in case they hit it)

@redshiftzero redshiftzero added this to the 0.1.0alpha milestone Oct 31, 2018
redshiftzero added a commit that referenced this issue Oct 31, 2018
@redshiftzero
Fix #83: Qubes file downloading and moving to the correct location
e636673
@redshiftzero redshiftzero mentioned this issue Oct 31, 2018
Merged
legoktm pushed a commit that referenced this issue Dec 11, 2023
@sssoleileraaa
Merge pull request #83 from freedomofpress/update-dev-dependencies
426fa2e 
Update dev requirements
legoktm pushed a commit that referenced this issue Dec 15, 2023
@redshiftzero
Merge pull request #83 from freedomofpress/creviera-patch-1
52504f4 
Update CHANGELOG.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@redshiftzero redshiftzero

Labels
bug Something isn't working qubes needed
Projects
None yet
Milestone
0.1.0alpha
Development

No branches or pull requests
2 participants
@heartsucker @redshiftzero