Attempting to backup / seems to result in nonsensical rsync command line

Bug #588841 reported by Mike.lifeguard
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Back In Time
Fix Released
High
Unassigned
backintime (Debian)
Fix Released
Undecided
Unassigned
backintime (Ubuntu)
Fix Released
High
Unassigned

Bug Description

Here's the log entry, showing a nonsensical rsync command line:

Jun 2 14:31:06 localhost backintime (root): INFO: Command "rsync -aEAX -v --delete-excluded --chmod=Fa-w,D+w --whole-file --delete --exclude="/var/backups" --exclude="/root/.local/share/backintime" --include="//" --exclude=".*" --exclude="*.backup*" --exclude="*~" --exclude="/home/mikelifeguard/backups" --exclude="/home/mikelifeguard/Downloads" --exclude="/home/mikelifeguard/Music" --exclude="/home/mikelifeguard/Pictures" --exclude="/home/mikelifeguard/Videos" --exclude="/var/backups" --exclude="/dev" --exclude="/cdrom" --exclude="/proc" --exclude="/tmp" --include="//**" --exclude="*" / "/var/backups/backintime/new_snapshot/backup/"" returns 0

ProblemType: Bug
DistroRelease: Ubuntu 10.04
Package: backintime-gnome 0.9.26-3ubuntu1
ProcVersionSignature: Ubuntu 2.6.32-22.33-generic 2.6.32.11+drm33.2
Uname: Linux 2.6.32-22-generic i686
Architecture: i386
CheckboxSubmission: 8b15f4fca5292e90a3327d5e528e925e
CheckboxSystem: 988b7c9be54a1e2525cae7d97805a134
Date: Wed Jun 2 14:32:04 2010
InstallationMedia: Ubuntu 9.10 "Karmic Koala" - Release i386 (20091028.5)
PackageArchitecture: all
ProcEnviron:
 PATH=(custom, user)
 LANG=en_CA.utf8
 SHELL=/bin/bash
SourcePackage: backintime

Revision history for this message
Mike.lifeguard (mikelifeguard) wrote :
Revision history for this message
Jonathan Wiltshire (jwiltshire) wrote :

Hi,

Please attach the file /home/mikelifeguard/.config/backintime/config which contains your BackInTime configuration for upstream to investigate.

Thanks.

Revision history for this message
Mike.lifeguard (mikelifeguard) wrote :
Revision history for this message
Dan (danleweb) wrote :

You have:

snapshots.exclude_patterns=.*:*.backup*:*~:/home/mikelifeguard/Videos:/home/mikelifeguard/Pictures:/home/mikelifeguard/Downloads:/home/mikelifeguard/backups:/home/mikelifeguard/Music

snapshots.include_folders=/home/mikelifeguard|0

I see no problem in rsync command. Can you explain what you find strage in it ?

Revision history for this message
Mike.lifeguard (mikelifeguard) wrote :

--exclude="/var/backups" is in there twice, and the following don't make sense to me:
--include="//"
--exclude=".*"
--include="//**"
--exclude="*"

Revision history for this message
jerico (jerico-dev) wrote :

I'm affected by the same problem. When trying to back up '/' the rsync command simply doesn't select any files for backup. The rsync command will only work if you leave out the include/exclude parts Mike.lifeguard mentioned.

Revision history for this message
jerico (jerico-dev) wrote :

BTW: I'll try to fix this and send you a debdiff.

Revision history for this message
jerico (jerico-dev) wrote :

The attached patch works for me. As it changes the logic for treating included dirs a bit it should be reviewed with care before general inclusion into the package.

Dan (danleweb)
Changed in backintime:
status: New → Fix Committed
importance: Undecided → High
tags: added: patch
removed: i386
Changed in backintime (Ubuntu):
importance: Undecided → High
assignee: nobody → Jonathan Wiltshire (debian-jwiltshire)
assignee: Jonathan Wiltshire (debian-jwiltshire) → nobody
Changed in backintime (Debian):
status: New → In Progress
Changed in backintime (Debian):
status: In Progress → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package backintime - 0.9.26-4

---------------
backintime (0.9.26-4) unstable; urgency=low

  * debian/control: depend on the menu packages for both GNOME and KDE
    front-ends, so we can use su-to-root on headless systems (LP:
    #466088)
  * Patch allow-root-backup.patch: allow backup of root directory,
    cherry-picked from upstream (LP: #588841) - thanks to jerico
  * Explicitly declare source format 1.0
  * Standards version 3.9.0 (no changes)
 -- Jonathan Wiltshire <email address hidden> Tue, 20 Jul 2010 16:38:22 +0100

Changed in backintime (Ubuntu):
status: New → Fix Released
Dan (danleweb)
Changed in backintime:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.