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

Fix --force, fix windows filename issue & introduce --ignore-write-errors #5

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

TheDelta
Copy link

First of all thanks for this great project ❤️, it helped me with client.

I found a couple of issues and fixed them:

  • --force is now working (missing variable)
  • if a linux backup is extraced on windows, there could be the issue (which I faced) that some filenames are invalid.
    • for example somelog-00:00.log and the whole extract process would fail
    • therefore I added a filename & filepath fix for windows to remove invalid characters
  • I also added a general --ignore-write-errors option in case there are some other issues.
    • This allows at least to get all other files except of the faulty ones

some trivial code refactoring and added .editorconfig

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.

1 participant