[Box Backup] Samba and Boxbackup

[Torsten Boob]

--MP_xL/uEywCX6_BkIYi3VBhEaD
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

On Fri, 15 Dec 2006 10:02:07 +0000
Ben Summers <ben@fluffy.co.uk> wrote:

> 
> On 14 Dec 2006, at 22:29, Torsten Boob wrote:
> 
> > Hi,
> >
> > i make my Backup using Linux as Client 1. Client 2 runs Linux and  
> > Samba. Exports from Client 2 are successfully mounted using Samba  
> > by Client 1 and backed up using boxbackup.
> >
> > If i restore Data, i always become an
> >
> > Exception: Common OSFileError (Error accessing a file. Check  
> > permissions.) (1/9).
> >
> > I think that happens while boxbackup tries to set up the  
> > permissions for the file just restored.
> >
> > Does anybody have an idea how to make that working ?
> >
> > I dont want to install boxbackup on every Computer, especially not  
> > on the Windows machines. So i want to have one Boxbackup Client  
> > that mounts the Directories from the Workstations and back it up.
> 
> 
> This isn't a mode Box Backup is designed to support, although it is  
> one which will be supported in the next version.
> 
> Having said that, compile up a debug version and then report back on  
> what that outputs. I do suspect, though, that it's trying to restore  
> the permissions on a file which isn't supported by Samba or you  
> haven't got permission to set.
> 

Restoring to NFS- or Samba-mounted storage makes problems while setting
file owner (chown). So i disabled throwing an exception if chown fails
(see attached file boxbackup_disable_chown_errors.patch).

You wrote, that the next version will support this mode. Does anybody work on it ?

Will it be possible to backup and restore ACL from windows exports ?



--MP_xL/uEywCX6_BkIYi3VBhEaD
Content-Type: text/x-patch; name=boxbackup_disable_chown_errors.patch
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment;
 filename=boxbackup_disable_chown_errors.patch

Index: lib/backupclient/BackupClientFileAttributes.cpp
===================================================================
--- lib/backupclient/BackupClientFileAttributes.cpp	(Revision 1)
+++ lib/backupclient/BackupClientFileAttributes.cpp	(Revision 2)
@@ -689,13 +689,13 @@
 				// Not a link, use normal chown
 				if(::chown(Filename, ntohl(pattr->UID), ntohl(pattr->GID)) != 0)
 				{
-					THROW_EXCEPTION(CommonException, OSFileError)
+//					THROW_EXCEPTION(CommonException, OSFileError)
 				}
 			}
 		#else
 			if(::lchown(Filename, ntohl(pattr->UID), ntohl(pattr->GID)) != 0)	// use the version which sets things on symlinks
 			{
-				THROW_EXCEPTION(CommonException, OSFileError)
+//				THROW_EXCEPTION(CommonException, OSFileError)
 			}
 		#endif
 	}

--MP_xL/uEywCX6_BkIYi3VBhEaD--