KurlesHS/restic
1
0
Fork 0
forked from TrueCloudLab/restic
Code Pull requests Activity

Improve error message for 'forget'

Browse source 
$ bin/restic forget /d 7 /w 4 /m 12
    argument "/d" is not a snapshot ID, ignoring
    argument "7" is not a snapshot ID, ignoring
    argument "/w" is not a snapshot ID, ignoring
    argument "4" is not a snapshot ID, ignoring
    argument "/m" is not a snapshot ID, ignoring
    cound not find a snapshot for ID "12", ignoring
This commit is contained in:
Alexander Neumann 2016-10-10 20:55:02 +02:00
parent 6f72164bbe
commit 250b36eeb1
1 changed files with 16 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
src/cmds/restic/cmd_forget.go
View file

@ -1,6 +1,7 @@
package main package main
import ( import (
"encoding/hex"
"fmt" "fmt"
"io" "io"
"restic" "restic"
@ -116,11 +117,24 @@ func runForget(opts ForgetOptions, gopts GlobalOptions, args []string) error {
return err return err
} }
// first, process all snapshot IDs given as arguments // parse arguments as hex strings
var ids []string
for _, s := range args { for _, s := range args {
_, err := hex.DecodeString(s)
if err != nil {
Warnf("argument %q is not a snapshot ID, ignoring\n", s)
continue
}
ids = append(ids, s)
}
// process all snapshot IDs given as arguments
for _, s := range ids {
id, err := restic.FindSnapshot(repo, s) id, err := restic.FindSnapshot(repo, s)
if err != nil { if err != nil {
return err Warnf("cound not find a snapshot for ID %q, ignoring\n", s)
continue
} }
if !opts.DryRun { if !opts.DryRun {