Merge lp:~dave2010/backintime/testhardlink into lp:backintime/1.0

[Dave]

Add a test to see if the target filesystem supports hardlinks. If it doesn't exit with failure since the point of Back In Time is to support incremental backups. Bug 581614.

[lp:~dave2010/backintime/testhardlink]

717. By Dave on 2010-07-10

Merge from trunk 727

Unmerged revisions

717. By Dave on 2010-07-10

Merge from trunk 727

716. By Dave on 2010-06-19

Merge from trunk 719

715. By Dave on 2010-05-27

Add test for hard link support

=== modified file 'common/snapshots.py'
--- common/snapshots.py	2010-07-06 13:32:31 +0000
+++ common/snapshots.py	2010-07-10 06:50:53 +0000
@@ -860,6 +860,13 @@
 				os.system( 'sleep 2' ) #max 1 backup / second
 				return [ False, True ]
 
+		test_path = self.config.get_snapshots_full_path()
+		if not tools.can_create_hardlinks( test_path ):
+			logger.error( "Backup filesystem does not support hard links.  rsync will be unable to create incremental backups on %s" % test_path )
+			self.set_take_snapshot_message( 1, _('Backup filesystem does not support hard links.  rsync will be unable to create incremental backups on %s') % test_path )
+			os.system( 'sleep 2' ) #max 1 backup / second
+			return [ False, True ]
+
 		new_snapshot_path_to = self.get_snapshot_path_to( new_snapshot_id )
 		
 		#create exclude patterns string

=== modified file 'common/tools.py'
--- common/tools.py	2010-07-04 09:28:54 +0000
+++ common/tools.py	2010-07-10 06:50:53 +0000
@@ -372,6 +372,27 @@
 			else:
 				raise
 
+def can_create_hardlinks( path ):
+	source_inode = 0
+	destination_inode = 0
+	source_path = os.path.join( path, 'tmptestfile-source' )
+	destination_path = os.path.join( path, 'tmptestfile-destination' )
+
+	try:
+		file = open( source_path, 'w' )
+		file.close()
+		source_inode = os.stat( source_path ).st_ino
+		cmd = "cp -l \"%s\" \"%s\"" % ( source_path, destination_path )
+		os.system( cmd )
+		destination_inode = os.stat( destination_path ).st_ino
+	except:
+		pass
+	os.system("rm -f \"%s\" \"%s\"" % ( source_path, destination_path ) )
+
+	if source_inode > 0 and source_inode == destination_inode:
+		return True
+
+	return False
 
 def _get_md5sum_from_path(path):
     '''return md5sum of path, af available system command md5sum()'''