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

Can't create zip archive on external disk when local disk has not enough space for temporary file #39

Open
davidak opened this issue Jun 5, 2023 · 2 comments

Comments

@davidak
Copy link

davidak commented Jun 5, 2023

What Happened?

I tried to create a zip archive on an SMB (Samba) share of a folder with 15 GB data.

Screenshot from 2023-06-05 16-21-50

Screenshot from 2023-06-05 16-11-44

It always fails with this unhelpful error.

Screenshot from 2023-06-05 16-15-17

I watched the free disk space on the local disk and it turns out, the zip archive is temporary created on the local disk and then moved, but the local disk does not have enough space free. It works when compressing a smaller folder.

https://www.youtube.com/watch?v=_gbEWEj7Kvc

Compressing this 15.3 GB folder needs about 25 GB temporary free disk space.

compress

The final zip archive on the SMB share is 13.7 GB and it looks like this much space is not freed on my local disk after compressing is finished!

Steps to Reproduce

  1. watch free disk space: watch -d -n 0.5 "df -h | grep /dev/dm-2 (use your device where / is mounted)
  2. right click folder in files
  3. Click "Compress"
  4. Select "Location" on an SMB share (or anywhere not on the / partition)
  5. Click "Create" button

Expected Behavior

features should work and errors should be helpful

always check if the space to create files is free. maybe not only for archives, but for any file save! could you implement that in granite?
investigate if the zip file could directly be created on the final location, so this still works when local disk is full
if you fail to find an elegant solution, at least give the user useful feedback, like "not enough space available on DISKNAME to create temporary archive file"
show in the dialog that the file is moved from temp location to final location. it's currently intransparent what is happening! https://youtu.be/KMllcb92XQ8?t=168

OS Version

Other Linux

NixOS 22.11.4021.54495a4eafe

Software Version

Older release (I have not run all updates)

elementary-files-6.2.1

Log Output

i did start files from terminal, but got no output

Hardware Info

Lenovo Thinkpad X230
Ethernet connection

@jeremypw
Copy link

jeremypw commented Jun 5, 2023

Files subcontracts file compression to FileRoller (https://github.com/elementary/fileroller) whence the error message comes so I think this issue should be moved there.

@jeremypw jeremypw transferred this issue from elementary/files Jun 5, 2023
@davidak davidak changed the title Can't create zip archive on different disk when local disk has not enough space for temporary file Can't create zip archive on external disk when local disk has not enough space for temporary file Jun 6, 2023
@Marukesu
Copy link
Contributor

Marukesu commented Jun 6, 2023

I believe the policy here is that patches applied here should be minimal, if this is a issue that comes from upstream, then it should be reported there. and we pick the fixes when a newer version get released.

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

No branches or pull requests

3 participants