--- sorry if this dupes, mail client issues... 
In my tests of using git (both Linus and pasky versions) I had a 
problem with  
doing "gitadd.sh *" where * expands to include directories.  This 
patch allows update-cache.c to more gracefully handle a directory 
passed into the add_file_to_cache method.  Without this patch 
exits prematurely with an error similar to: 
fatal: Unable to add plx to database 
Now it spits out: 
'plx' is a directory, ignoring 
Which from an end user stand point is better. 
BTW, so far my tests of using git are positive for my small Dreamcast 
projects...  I was previously using subversion but find it to be a bit 
overkill for these small projects. 
Martin Schlemmer,  I ran "emerge sync" today and found git has been 
added to  
portage, version 0.5.  Also note that there are now two "git" entries 
portage app-misc/git and dev-util/git.  app-misc/git is GNU 
Interactive Tools 
Signed-off-by: Mike Dunston ([EMAIL PROTECTED]) 
Index: update-cache.c 
--- 690494557d393ca78f69a8569880ed4a3aeda276/update-cache.c  
+++ uncommitted/update-cache.c  (mode:100644) 
@@ -104,6 +104,11 @@ 
                return -1; 
+       if(S_ISDIR(st.st_mode)) { 
+               printf("'%s' is a directory, ignoring\n", path); 
+               close(fd); 
+               return 0; 
+       } 
        namelen = strlen(path); 
        size = cache_entry_size(namelen); 
        ce = malloc(size); 
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to