Skip to content

When there is an error while compressing, original files should not be sent to the Bin #885

Closed

Description

@victor

I encountered a disk full error when compressing some folders, with the option to move the original files to the Bin after compression. This option should only be honored on successful compression, to avoid data loss and an easier retry.

One could argue that the files in the Bin are not lost and the can be recovered easily, but that is not entirely true: there may be many similarly named files in the bin and finding the correct ones may prove complicated. Also, one of the first things one can do irreflexively (meaning, I did it) is to empty the Bin, to gain disk space. Doing this ensures there's no trace of the original source files to retry compression after gaining some disk space

Activity

  1. aonez commented on Jul 2, 2021

    @aonez
    Owner

    It's set to only remove files if the operation works properly. I'll test if a disk full error bypasses somehow this.

  2. self-assigned this
    on Jul 2, 2021
  3. added this to the Look at milestone on Jul 2, 2021
  4. aonez commented on Jul 2, 2021

    @aonez
    Owner

    @victor what format and setting were you using?

  5. reopened this on Jul 11, 2021
  6. reopened this on Jul 11, 2021
  7. modified the milestones: Look at, 1.2.16 on Jul 11, 2021
  8. victor commented on Jul 12, 2021

    @victor
    Author

    Format was 7z, with slow, max compression.

  9. Repository owner deleted a comment from no-response on Jul 12, 2021
  10. Repository owner deleted a comment from no-response on Jul 12, 2021
  11. aonez commented on Jul 12, 2021

    @aonez
    Owner

    I've detected the issue, fixed for next revision, coming in a few. Thanks a lot @victor!

  12. victor commented on Jul 12, 2021

    @victor
    Author
  13. aonez commented on Jul 15, 2021

    @aonez
    Owner

    @victor, as a followup:

    This should be fixed in the upcoming 1.2.16. Test build: Keka-1.2.16.r4608.7z

    There was an issue detecting the error codes that has been reported in various ways (#891 #885). Thanks a lot for the feedback!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions