-
-
Notifications
You must be signed in to change notification settings - Fork 750
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
No dry-run flags when using borg extract #8564
Comments
Good idea. It's just questionable if we shall do that for 1.4 or rather put it into 2.0 breaking release. Not sure if somebody is parsing that output with some automation which would break if we change the format. |
I just reported this for borg 1.4, since I found this inconsistency using this version. |
I shortly looked into this - but unfortunately fixing this is beyond my skills and knowledge. I could not even find the relevant places in the source code. :-D Hopefully somebody more proficient will take this up. |
The starting point for changing how some command works is its implementation in |
Hi! First time contributor here. Could you please give me some advice on getting started? I would like to start with this as my first problem! |
@alighazi288 Thanks for helping! Did you read the previous comment, where I pointed to the About general borg installation, development setup, etc. please see our docs. If you have more specific questions, just ask! |
Thank you so much! I'll try working on a fix for the problem! |
@alighazi288 When testing @ThomasWaldmann Is there a conceptional difference in what |
There is a massive difference in create and extract dry-run:
|
Using borg 1.4, I checked the use of "flags" when doing a dry-run for different borg commands.
The command
borg extract --dry-run --list
does not print the flags (that are used with e.g. borg create / borg recreateInstead,
borg extract
prints only the list of items that would be included in the extract without the --dry-run.This was the command, I used:
borg extract REPO::ARCHIVE /home/user/test Admin -e 'home/user/test/*.bak' --dry-run --verbose --list
In the name of consistency, dry-run flags could be added to borg extract.
The text was updated successfully, but these errors were encountered: