Package: backupninja
Version: 0.9.3-6
Severity: normal
Tags: patch

If I point the 'src' config option to to a trac repo directly (instead of to
the parent directory) then the trac-admin hotcopy command fails with the
following error:

  Debug: Command failed: [Errno 17] File exists: '/var/backups/trac.tmp/.'
  Hotcopying /home/trac to /var/backups/trac.tmp/. ...
  Error: command failed -- trac-admin /home/trac/. hotcopy 
/var/backups/trac.tmp/.
  Error: because of earlier errors, we are leaving trac backups in 
/var/backups/trac.tmp instead of /var/backups/trac

The trailing '.' is confusing trac-admin. The attached patch catches this
special case.

The reason for not wanting to point 'src' to the parent directory is that I
don't want it to search the whole disk when I know where the single trac
installation is located (the worst-case scenario is when the repo is '/trac').

-- System Information:
Debian Release: testing/unstable
  APT prefers testing
  APT policy: (990, 'testing'), (500, 'unstable')
Architecture: i386 (i686)
Shell:  /bin/sh linked to /bin/bash
Kernel: Linux 2.6.15-1-686
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)

Versions of packages backupninja depends on:
ii  bash                      3.1-2          The GNU Bourne Again SHell
ii  dialog                    1.0-20060221-1 Displays user-friendly dialog boxe
ii  mawk                      1.3.3-11       a pattern scanning and text proces

backupninja recommends no packages.

-- no debconf information
--- /usr/share/backupninja/trac.orig    2006-04-08 14:03:53.000000000 -0700
+++ /usr/share/backupninja/trac 2006-08-12 04:53:43.000000000 -0700
@@ -14,6 +14,7 @@
 for repo in `find . -name VERSION`
 do
     repo=`dirname $repo`
+    if [ "$repo" == "." ]; then repo=""; fi
 
     # Just make the parent directory for $tmp/$repo
     parentdir=`dirname $tmp/$repo`

Reply via email to