X-Git-Url: http://git.roojs.org/?a=blobdiff_plain;f=GitBranch.vala;h=c08721be57d61f4f40d04258b30c13585676d8f0;hb=556661ee18a4b114d63e652935773a54362a4af1;hp=c7d3921481cb51727b3bc2de36c1a4a5eac9c2ac;hpb=ff414b16c9e9e9827a5e7e508b817485662c0e23;p=gitlive

diff --git a/GitBranch.vala b/GitBranch.vala
index c7d39214..c08721be 100644
--- a/GitBranch.vala
+++ b/GitBranch.vala
@@ -85,18 +85,31 @@ public class GitBranch : Object
 			remotes.set(rn.substring(11), rh.get_oid());
 			remotes_used.set(rn.substring(11), false);
 
-	  }
+	  } 
 		
 		 
 		var r = repo.repo.enumerate_branches(Ggit.BranchType.LOCAL);
 		while (r.next()) {
+		
+
+		
 			var br = new GitBranch(repo);
 			var gbr = r.get() as Ggit.Branch;
-			repo.branches.set(gbr.get_name(), br);
+			if (!gbr.is_branch()) {
+				continue;
+			}
+			
 			br.active = gbr.is_head();
 			br.name = gbr.get_name();
 			br.lastrev = gbr.get_target().to_string();
-			var rname = gbr.get_upstream() != null ? gbr.get_upstream().get_name() : "";
+			string rname ;
+			try {
+				rname = gbr.get_upstream() != null ? gbr.get_upstream().get_name() : "";
+			} catch(Error e) {
+				GLib.debug("Skip branch = got error");
+				continue;
+			}
+			repo.branches.set(gbr.get_name(), br);			
 			if (rname.has_prefix("refs/remotes/origin/")) {
 				rname = rname.substring(20);
 				if (remotes.has_key(rname)) {
@@ -104,21 +117,32 @@ public class GitBranch : Object
 					br.remoterev = remotes.get(rname).to_string();
 					remotes_used.set(rname,true);
 					size_t ahead, behind;
-					repo.repo.get_ahead_behind(
-						gbr.get_target(),
-						remotes.get(rname),
-						out ahead,
-						out behind
-					);
-					br.ahead = (int)ahead;
-					br.behind = (int) behind;
+					br.behind = 1;
+					
+					try {
+						repo.repo.get_ahead_behind(
+							gbr.get_target(),
+							remotes.get(rname),
+							out ahead,
+							out behind
+						);
+						br.ahead = (int)ahead;
+						br.behind = (int) behind;
 
+					} catch(Error e) {
+						GLib.debug("we probably need to fetch... %s", repo.name);
+					}
+					
 				}
 				 
 				
 				// age?
 				// behind or infront..
 			}
+			if (br.active) {
+				GLib.debug("repo: %s currentBranch = %s", repo.name, br.name);
+				repo._currentBranch = br;
+			}
 			
 		}
 		
@@ -127,20 +151,35 @@ public class GitBranch : Object
 			if (remotes_used.get(rn)) {
 				continue;
 			}
+			if (repo.branches.has_key(rn)) {
+				GLib.debug("skip tracking branch - same name exists?");
+				continue;
+			}
 			// not clear how to do this yet...
-			repo.git(  { "branch" ,"--track" , rn,  "origin/" + rn} ); 
+			try {
+				repo.git(  { "branch" ,"--track" , rn,  "origin/" + rn} ); 
+			} catch (Error e) {
+				continue; // allow failure?
+			}
 			var br = new GitBranch(repo);
 			br.name = rn;
-			br.remoterev  = br.lastrev = remotes.get(rn).to_string();
+			br.lastrev = ""; // it's behind
+			br.remoterev  =  remotes.get(rn).to_string();
 			br.remote = rn;
+			br.ahead = 0;
+			br.behind = 1;
 			// behind/ahead == 0...
 			repo.branches.set(rn, br);
 			
 		}
+
+		if (repo._currentBranch == null) {
+			GLib.error("could not find active Branch for %s", repo.name);
+		}
 		
+		         
 		
-		
-       	
+       	/*
       //	 repo.git( { "fetch",  "-a" } ); == done async before...
        	
      	string[] cmd = { "branch",   "--no-color", "--verbose", "--no-abbrev" , "-a"  };
@@ -211,7 +250,7 @@ public class GitBranch : Object
         	br.name  = newname;
         	local.set(br.name, br);
         }
-        
+        */
         /*
         this bit of the code tries to turn a local branch into a track of a remote one.
         -- that's probably not a good idea.
@@ -226,9 +265,9 @@ public class GitBranch : Object
         	name = name.replace("/", ".");
         	
         }
-        */
+       
         repo.branches = local;
-        
+         */
     
 	
 	}
@@ -260,10 +299,7 @@ public class GitBranch : Object
 		return this.is_remote ? this.remote : this.name;
 	}
 	
-	public void create()
-	{
-	
-	}
+//	public void create() { }
 	
 }