diff options
Diffstat (limited to 'doStats.go')
| -rw-r--r-- | doStats.go | 66 |
1 files changed, 56 insertions, 10 deletions
@@ -15,11 +15,12 @@ import ( ) func doVerify() (string, error) { - if argv.Verify.All { + var allerr error + if argv.Verify.All != nil { for r := range me.forge.Repos.IterByFullPath() { if r.Stats == nil { - doStats(r) - return r.FullPath, nil + _, err := doStats(r) + allerr = errors.Join(allerr, err) } } return "verify ran everywhere", nil @@ -30,26 +31,52 @@ func doVerify() (string, error) { return "no repo", errors.New("working dir isn't a repo I know about") } - return doStats(repo) + s, err := doStats(repo) + allerr = errors.Join(allerr, err) + + user, err := refHash(repo, "heads/jcarr") + allerr = errors.Join(allerr, err) + master, err := refHash(repo, "remotes/origin/master") + allerr = errors.Join(allerr, err) + HEAD, err := refHash(repo, "remotes/origin/HEAD") + allerr = errors.Join(allerr, err) + log.Printf("user=%10.10s master=%10.10s HEAD=%10.10s\n", user, master, HEAD) + safeDelete(repo, user, HEAD) // delete user if safely contained in HEAD + + return s, allerr } func doStats(r *gitpb.Repo) (string, error) { var allerr error pb, err := r.LoadStats() + if err == nil { + log.Info("LoadStats() ok", pb.Filename) + } else { + log.Info("LoadStats() err", err) + } allerr = errors.Join(allerr, err) if hasOrigin(r) { - log.Info("repo doesn't have origin") + log.Info("todo: detect origin") } // collect the stats counter, err := last100(r, pb) allerr = errors.Join(allerr, err) s := log.Sprintf("found %d new hashes", counter) - if counter > 0 { - pb.Save() + + for stat := range pb.IterAll() { + if stat.PatchId == "" { + stat.PatchId, err = r.FindPatchIdByHash(stat.Hash) + allerr = errors.Join(allerr, err) + log.Info("patchid for hash", stat.Hash, "is", stat.PatchId) + counter += 1 + } } + if counter > 0 { + pb.SaveVerbose() + } // build() return s, nil } @@ -58,9 +85,28 @@ func collectStats(r *gitpb.Repo, pb *gitpb.Stats) error { return nil } -// git show-ref --verify refs/remotes/origin/HEAD +// git show-ref refs/heads/devel +func refHash(r *gitpb.Repo, name string) (string, error) { + var hash string + refname := "refs/" + name + cmd := []string{"git", "show-ref", refname} + cmdout := r.Run(cmd) + for i, line := range cmdout.Stdout { + parts := strings.Fields(line) + if config.If("stats") { + log.Info(parts[0], "LINE:", i, line) + } + hash = parts[0] + } + if len(cmdout.Stdout) != 1 { + return "", errors.New("no refname:" + name) + } + return hash, nil +} + +// git show-ref --verify refs/heads/devel func hasOrigin(r *gitpb.Repo) bool { - // git show-ref refs/remotes/origin/HEAD + // git show-ref refs/heads/devel return true } @@ -80,7 +126,7 @@ func makeFmts() string { func last100(r *gitpb.Repo, pb *gitpb.Stats) (int, error) { var allerr error var counter int - cmd := []string{"git", "log", "-n", "10", makeFmts(), "origin/" + r.GetMasterBranchName()} + cmd := []string{"git", "log", "-n", "100", makeFmts(), "origin/" + r.GetMasterBranchName()} if config.If("stats") { log.Info("Run:", cmd) } |