]> Cypherpunks.ru repositories - goredo.git/blobdiff - ood.go
Collapse lines
[goredo.git] / ood.go
diff --git a/ood.go b/ood.go
index dffaa4ef06b8ea1335e51636751a880d0191cbe7..c11b23f5995ab89ef95089477828429e5df00cc0 100644 (file)
--- a/ood.go
+++ b/ood.go
@@ -20,7 +20,6 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
 package main
 
 import (
-       "bytes"
        "errors"
        "fmt"
        "io"
@@ -49,6 +48,7 @@ var (
 
        OODCache        = make(map[string]bool)
        FileExistsCache = make(map[string]bool)
+       DepInfoCache    = make(map[string]*DepInfo)
 
        ErrMissingTarget = errors.New("invalid format of .rec: missing Target")
 )
@@ -100,81 +100,86 @@ func isOODByBuildUUID(tgt *Tgt) bool {
 func isOOD(tgt *Tgt, level int, seen map[string]*Tgt) (bool, error) {
        indent := strings.Repeat(". ", level)
        tracef(CDebug, "ood: %s%s checking", indent, tgt)
-       ood, cached := OODCache[tgt.a]
+       ood, cached := OODCache[tgt.rel]
        if cached {
                tracef(CDebug, "ood: %s%s -> cached: %v", indent, tgt, ood)
                return ood, nil
        }
-       depInfo, err := depRead(tgt)
-       if err != nil {
-               if errors.Is(err, fs.ErrNotExist) {
-                       if isSrc(tgt) {
-                               ood = false
-                               tracef(CDebug, "ood: %s%s -> is source", indent, tgt)
-                       } else {
-                               ood = true
-                               tracef(CDebug, "ood: %s%s -> no dep: %s", indent, tgt, tgt.Dep())
-                       }
-                       OODCache[tgt.a] = ood
-                       return ood, nil
-               }
+       dep := DepCache[tgt.rel]
+       var err error
+       if dep == nil {
+               dep, err = depRead(tgt)
                if err != nil {
-                       return true, TgtError{tgt, ErrLine(err)}
+                       if errors.Is(err, fs.ErrNotExist) {
+                               if isSrc(tgt) {
+                                       ood = false
+                                       tracef(CDebug, "ood: %s%s -> is source", indent, tgt)
+                               } else {
+                                       ood = true
+                                       tracef(CDebug, "ood: %s%s -> no dep: %s", indent, tgt, tgt.Dep())
+                               }
+                               OODCache[tgt.rel] = ood
+                               return ood, nil
+                       }
+                       if err != nil {
+                               return true, TgtError{tgt, ErrLine(err)}
+                       }
                }
+               DepCache[tgt.rel] = dep
        }
 
-       if depInfo.build == BuildUUID {
+       if dep.build == BuildUUID {
                tracef(CDebug, "ood: %s%s -> already built", indent, tgt)
-               OODCache[tgt.a] = false
+               OODCache[tgt.rel] = false
                return false, nil
        }
        if !FileExists(tgt.a) {
                tracef(CDebug, "ood: %s%s -> non-existent", indent, tgt)
-               OODCache[tgt.a] = true
+               OODCache[tgt.rel] = true
                return true, nil
        }
 
-       for _, dep := range depInfo.ifcreates {
-               if FileExists(dep.a) {
-                       tracef(CDebug, "ood: %s%s -> %s created", indent, tgt, dep)
+       for _, ifcreate := range dep.ifcreates {
+               if FileExists(ifcreate.a) {
+                       tracef(CDebug, "ood: %s%s -> %s created", indent, tgt, ifcreate)
                        ood = true
                        goto Done
                }
        }
 
-       for _, dep := range depInfo.ifchanges {
-               tracef(CDebug, "ood: %s%s -> %s: checking", indent, tgt, dep.tgt)
-               ood, cached = OODCache[dep.tgt.a]
+       for _, ifchange := range dep.ifchanges {
+               tracef(CDebug, "ood: %s%s -> %s: checking", indent, tgt, ifchange.tgt)
+               ood, cached = OODCache[ifchange.tgt.rel]
                if cached {
-                       tracef(CDebug, "ood: %s%s -> %s: cached: %v", indent, tgt, dep.tgt, ood)
+                       tracef(CDebug, "ood: %s%s -> %s: cached: %v", indent, tgt, ifchange.tgt, ood)
                        if ood {
                                goto Done
                        }
                        continue
                }
 
-               inode, err := inodeFromFileByPath(dep.tgt.a)
+               inode, err := inodeFromFileByPath(ifchange.tgt.a)
                if err != nil {
                        if errors.Is(err, fs.ErrNotExist) {
-                               tracef(CDebug, "ood: %s%s -> %s: not exists", indent, tgt, dep.tgt)
+                               tracef(CDebug, "ood: %s%s -> %s: not exists", indent, tgt, ifchange.tgt)
                                ood = true
-                               OODCache[dep.tgt.a] = ood
+                               OODCache[ifchange.tgt.rel] = ood
                                goto Done
                        }
                        return ood, TgtError{tgt, ErrLine(err)}
                }
 
                if inode.Size != dep.inode.Size {
-                       tracef(CDebug, "ood: %s%s -> %s: size differs", indent, tgt, dep.tgt)
+                       tracef(CDebug, "ood: %s%s -> %s: size differs", indent, tgt, ifchange.tgt)
                        ood = true
-                       OODCache[dep.tgt.a] = ood
+                       OODCache[dep.tgt.rel] = ood
                        goto Done
                }
                if InodeTrust != InodeTrustNone && inode.Equals(dep.inode) {
-                       tracef(CDebug, "ood: %s%s -> %s: same inode", indent, tgt, dep.tgt)
+                       tracef(CDebug, "ood: %s%s -> %s: same inode", indent, tgt, ifchange.tgt)
                } else {
-                       tracef(CDebug, "ood: %s%s -> %s: inode differs", indent, tgt, dep.tgt)
-                       fd, err := os.Open(dep.tgt.a)
+                       tracef(CDebug, "ood: %s%s -> %s: inode differs", indent, tgt, ifchange.tgt)
+                       fd, err := os.Open(ifchange.tgt.a)
                        if err != nil {
                                return ood, TgtError{tgt, ErrLine(err)}
                        }
@@ -183,46 +188,46 @@ func isOOD(tgt *Tgt, level int, seen map[string]*Tgt) (bool, error) {
                        if err != nil {
                                return ood, TgtError{tgt, ErrLine(err)}
                        }
-                       if !bytes.Equal(dep.hash, hsh) {
-                               tracef(CDebug, "ood: %s%s -> %s: hash differs", indent, tgt, dep.tgt)
+                       if dep.hash != hsh {
+                               tracef(CDebug, "ood: %s%s -> %s: hash differs", indent, tgt, ifchange.tgt)
                                ood = true
-                               OODCache[dep.tgt.a] = ood
+                               OODCache[ifchange.tgt.rel] = ood
                                goto Done
                        }
-                       tracef(CDebug, "ood: %s%s -> %s: same hash", indent, tgt, dep.tgt)
+                       tracef(CDebug, "ood: %s%s -> %s: same hash", indent, tgt, ifchange.tgt)
                }
 
-               if dep.tgt.a == tgt.a {
-                       tracef(CDebug, "ood: %s%s -> %s: same target", indent, tgt, dep.tgt)
+               if ifchange.tgt.rel == tgt.rel {
+                       tracef(CDebug, "ood: %s%s -> %s: same target", indent, tgt, ifchange.tgt)
                        continue
                }
-               if isSrc(dep.tgt) {
-                       tracef(CDebug, "ood: %s%s -> %s: is source", indent, tgt, dep.tgt)
-                       OODCache[dep.tgt.a] = false
+               if isSrc(ifchange.tgt) {
+                       tracef(CDebug, "ood: %s%s -> %s: is source", indent, tgt, ifchange.tgt)
+                       OODCache[ifchange.tgt.rel] = false
                        continue
                }
 
-               if _, ok := seen[dep.tgt.a]; ok {
-                       tracef(CDebug, "ood: %s%s -> %s: was always built", indent, tgt, dep.tgt)
-                       OODCache[dep.tgt.a] = false
+               if _, ok := seen[ifchange.tgt.rel]; ok {
+                       tracef(CDebug, "ood: %s%s -> %s: was always built", indent, tgt, ifchange.tgt)
+                       OODCache[ifchange.tgt.rel] = false
                        continue
                }
 
-               depOOD, err := isOODWithTrace(dep.tgt, level+1, seen)
+               depOOD, err := isOODWithTrace(ifchange.tgt, level+1, seen)
                if err != nil {
                        return ood, TgtError{tgt, err}
                }
                if depOOD {
-                       tracef(CDebug, "ood: %s%s -> %s: ood", indent, tgt, dep.tgt)
+                       tracef(CDebug, "ood: %s%s -> %s: ood", indent, tgt, ifchange.tgt)
                        ood = true
                        goto Done
                }
-               tracef(CDebug, "ood: %s%s -> %s: !ood", indent, tgt, dep.tgt)
+               tracef(CDebug, "ood: %s%s -> %s: !ood", indent, tgt, ifchange.tgt)
        }
 
 Done:
        tracef(CDebug, "ood: %s%s: %v", indent, tgt, ood)
-       OODCache[tgt.a] = ood
+       OODCache[tgt.rel] = ood
        return ood, nil
 }