Ticket.bjs
[gitlive] / GitBranch.vala
index d177cfd..a30808b 100644 (file)
@@ -36,6 +36,18 @@ public class GitBranch : Object
        public string remote = "";
        public string remoterev = "";
        
+       public string toString()
+       {
+               return 
+                       "active: " + (active ? "true" : "false") + "\n" +
+                       "is_remote: " + (is_remote ? "true" : "false") + "\n" +                 
+                       "lastrev: " + lastrev + "\n" +
+                       "name: " + name + "\n" +
+                       "remote: " + remote + "\n" +
+                       "remoterev: " + remoterev + "\n";
+       }
+       
+       
        public GitBranch(GitRepo repo)
        {
                this.repo = repo;
@@ -46,8 +58,11 @@ public class GitBranch : Object
                repo.branches = new Gee.HashMap<string,GitBranch>();
        
        var branches =  new Gee.HashMap<string,GitBranch>();
-       
+               var local =  new Gee.HashMap<string,GitBranch>();
+               var remotes  =  new Gee.HashMap<string,GitBranch>();
+               
+      //        repo.git( { "fetch",  "-a" } ); == done async before...
+               
        string[] cmd = { "branch",   "--no-color", "--verbose", "--no-abbrev" , "-a"  };
         var res = repo.git( cmd );
         var lines = res.split("\n");
@@ -57,6 +72,11 @@ public class GitBranch : Object
                        continue;
                }
                GLib.debug("add branch %s", br.realName());
+               if (!br.is_remote) {
+                       local.set(br.name, br);
+               } else {
+                       remotes.set(br.remote, br);
+                       }
                 
                branches.set(br.realName(), br);
                if (br.active) {
@@ -64,33 +84,53 @@ public class GitBranch : Object
                }
         }
         
-         var bl = repo.git(
+         var bl = repo.git({
                 "for-each-ref",
                 "--format",   "%(refname:short):remotes/%(upstream:short)",
                 "refs/heads"
-               ).split("\n");
+               }).split("\n");
         
-        for(var line in bl) {
+        foreach(var line in bl) {
                if (line.length < 1) continue;
                
                var ar = line.split(":remotes/");
                var lname= ar[0];
-            var rname = "remotes/' + ar[1]";
+            var rname = "remotes/" + ar[1];
+            
                 //print(rname);
                 // we should always have a local version of it.
-            branches.get(lname).remote = rname;
+            if (branches.has_key(lname)) {     
+                   branches.get(lname).remote = rname;
+            }
             
             if (!branches.has_key(rname)) {
                     continue;
             }
-            branches.get(name).remoterev =  branches.get(rname).lastrev;
+            branches.get(lname).remoterev =  branches.get(rname).lastrev;
                 // flag it for not adding..
                 
             branches.get(rname).name = lname;
         }
+        foreach(var br in local.values) {
+               GLib.debug("BRANCH:\n%s\n" , br.toString());
+        }
         
-        
-        repo.branches = branches;
+        /*
+        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.
+        var r_replace = new Regex("^remotes/");
+        var o_replace = new Regex("^origin/");
+        foreach(var r in remotes.values) {
+               if (r.name.length >0) {
+                       return;
+               }
+               var name = r_replace.replace(r.name, r.name.length,0, "");
+               name = o_replace.replace(name, name.length,0, "");
+               name = name.replace("/", ".");
+               
+        }
+        */
+        repo.branches = local;