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

Possible bug involving INCLIST and trailing forward-slashes #113

Open
kvnvallis opened this issue Jan 28, 2016 · 5 comments
Open

Possible bug involving INCLIST and trailing forward-slashes #113

kvnvallis opened this issue Jan 28, 2016 · 5 comments

Comments

@kvnvallis
Copy link

duplicity-backup.conf.example contains the following example for backing up a single location:

INCLIST=( "/home/foobar_user_name/Documents/" )

But in practice, leaving the trailing forward-slash on the folder you want to back up causes only the folders to be backed up, and not their contents. Duplicity-backup.sh will only list the folders inside Documents/ when the -l flag is used, and none of the actual files can be restored. I noticed this issue when the backup of my Pictures folder was not even a megabyte large.

I might be using an old version of duplicity-backup.sh, but the script doesn't seem to output a version number so I'm not sure how to check. Removing the trailing forward-slash solves the problem. Is this a bug?

@arichiardi
Copy link

I think this can be related with the way duplicity handles file selection

@zertrin
Copy link
Owner

zertrin commented Sep 4, 2016

Yeah, I'm not sure myself, I find the explanations about file selection quite hard to understand in the official documentation of duplicity. And don't see anything specifically speaking about trailing slashes in paths.

In any case, this issue is probably not caused by our script itself, since the paths given in INCLIST and EXCLIST are passed as is to duplicity.

I guess we should remove the final slash from the example. Not sure I can do more, but I'm open to suggestions.

@arichiardi
Copy link

arichiardi commented Sep 6, 2016

I can confirm the problem and the workaround, just add the option to static ones (with a newer duplicity):

STATIC_OPTIONS="--full-if-older-than 30D --exclude-filelist ~/.backup/HomeBackup.duplicity"

@pcoultha
Copy link

This Duplicity behavior is being tracked and discussed here:
https://bugs.launchpad.net/duplicity/+bug/1624725

It's also worth looking at this earlier request to re-introduce this behavior here:
https://bugs.launchpad.net/duplicity/+bug/1479545

@zertrin
Copy link
Owner

zertrin commented Sep 29, 2016

Thanks for the heads-up!

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

No branches or pull requests

4 participants