cmd: added clear_cache method now used by test repo decorators to be sure persistent commands are killed before trying to remove the directory. Unfortunately, it still claims someone has opened the file. handle.exe does not show anyone, so what is happening here ? Is it just a windows odity ? If nothing helps I could just keep the temp data, but lets do some more testing first

Author: Byron

TODO

@@ -11,9 +11,13 @@ General
   a sha or ref unless cat-file is used where it must be a sha
 * Overhaul command caching - currently its possible to create many instances of 
   the std-in command types, as it appears they are not killed when the repo gets 
-  deleted. A clear() method could already help to allow long-running programs
-  to remove cached commands after an idle time.
-
+  deleted.
+* git command on windows may be killed using the /F options which probably is like
+  SIGKILL. This is bad as the process might be doing something, leaving the resource
+  locked and/or in an inconsistent state. Without the /F option, git does not terminate 
+  itself, probably it listens to SIGINT which is not sent by TASKKILL. We can't really
+  help it, but may be at some point git handles windows signals properly.
+  
 Object
 ------
 * DataStream method should read the data itself. This would be easy once you have 

lib/git/cmd.py

@@ -17,7 +17,7 @@
 				  'output_stream' )
 
 extra = {}
-# NOTE: Execution through a shell appears to be slightly faster, but in fact
+# NOTE: Execution through a shell on windows appears to be slightly faster, but in fact
 # I consider it a problem whenever complex strings are passed and *interpreted* 
 # by the shell beforehand. This can cause great confusion and reduces compatability
 # between the OS which is why the shell should not be used ( unless it does not work
@@ -401,3 +401,16 @@ def get_object_data(self, ref):
 		cmd.stdout.read(1)		# finishing newlines
 
 		return (hexsha, typename, size, data)
+		
+	def clear_cache(self):
+		"""
+		Clear all kinds of internal caches to release resources.
+		
+		Currently persistent commands will be interrupted.
+		
+		Returns
+			self
+		"""
+		self.cat_file_all = None
+		self.cat_file_header = None
+		return self

test/testlib/helper.py

@@ -82,6 +82,7 @@ def bare_repo_creator(self):
 		try:
 			return func(self, rw_repo)
 		finally:
+			rw_repo.git.clear_cache()
 			shutil.rmtree(repo_dir)
 		# END cleanup
 	# END bare repo creator
@@ -107,6 +108,7 @@ def repo_creator(self):
 			try:
 				return func(self, rw_repo)
 			finally:
+				rw_repo.git.clear_cache()
 				shutil.rmtree(repo_dir)
 			# END cleanup
 		# END rw repo creator
@@ -179,6 +181,8 @@ def remote_repo_creator(self):
 			try:
 				return func(self, rw_repo, rw_remote_repo)
 			finally:
+				rw_repo.git.clear_cache()
+				rw_remote_repo.git.clear_cache()
 				shutil.rmtree(repo_dir)
 				shutil.rmtree(remote_repo_dir)
 			# END cleanup