apply patches
This commit is contained in:
parent
6d7d74feb4
commit
e0520ca96d
39
doPatch.go
39
doPatch.go
|
@ -101,21 +101,24 @@ func doPatch() error {
|
|||
|
||||
if argv.Patch.List != nil {
|
||||
var changed bool
|
||||
newpatches := forgepb.NewPatches()
|
||||
newpatches := new(forgepb.Set)
|
||||
newpatches.Patches = forgepb.NewPatches()
|
||||
for pset := range me.forge.Patchsets.IterAll() {
|
||||
pset.PrintTable()
|
||||
for patch := range pset.Patches.IterAll() {
|
||||
if err := setNewCommitHash(patch); err != nil {
|
||||
log.Infof("%s bad check on patch failure %v\n", patch.Filename, err)
|
||||
return err
|
||||
}
|
||||
changed = true
|
||||
if patch.NewHash == "" {
|
||||
if newpatches.AppendByPatchId(patch) {
|
||||
if patch.NewHash == "" || patch.NewHash == "na" {
|
||||
if newpatches.Patches.AppendByPatchId(patch) {
|
||||
log.Info("patchId added here", patch.PatchId)
|
||||
} else {
|
||||
log.Info("patchId already here", patch.PatchId)
|
||||
}
|
||||
} else {
|
||||
if err := setNewCommitHash(patch); err != nil {
|
||||
log.Infof("%s bad check on patch failure %v\n", patch.Filename, err)
|
||||
return err
|
||||
}
|
||||
log.Info("newhash set already here", patch.PatchId, patch.NewHash)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
@ -124,10 +127,14 @@ func doPatch() error {
|
|||
log.Warn("savePatchsets() failed", err)
|
||||
}
|
||||
}
|
||||
log.Info("NEW PATCHES TABLE")
|
||||
newpatches.PrintTable()
|
||||
if newpatches.Len() != 0 {
|
||||
for patch := range newpatches.IterAll() {
|
||||
log.Info("new patch:", patch.CommitHash, patch.NewHash, patch.Filename)
|
||||
for patch := range newpatches.Patches.IterAll() {
|
||||
if err := setNewCommitHash(patch); err == nil {
|
||||
log.Info("newhash set already here", patch.PatchId, patch.NewHash)
|
||||
continue
|
||||
}
|
||||
log.Infof("%s is new\n", patch.Filename)
|
||||
repo := me.forge.FindByGoPath(patch.Namespace)
|
||||
if repo == nil {
|
||||
log.Info("\tCould not find namespace:", patch.Namespace)
|
||||
|
@ -138,6 +145,17 @@ func doPatch() error {
|
|||
log.Info("apply results:", newhash, err)
|
||||
}
|
||||
}
|
||||
return nil
|
||||
/*
|
||||
if newpatches.Len() != 0 {
|
||||
for patch := range newpatches.IterAll() {
|
||||
log.Info("new patch:", patch.CommitHash, patch.NewHash, patch.Filename)
|
||||
repo := me.forge.FindByGoPath(patch.Namespace)
|
||||
if repo == nil {
|
||||
log.Info("\tCould not find namespace:", patch.Namespace)
|
||||
continue
|
||||
}
|
||||
}
|
||||
return log.Errorf("patches need to be applied")
|
||||
}
|
||||
|
||||
|
@ -156,6 +174,7 @@ func doPatch() error {
|
|||
}
|
||||
newpb.PrintTable()
|
||||
return nil
|
||||
*/
|
||||
}
|
||||
|
||||
// if nothing, show patches & dirty repos
|
||||
|
|
|
@ -181,23 +181,20 @@ func setNewCommitHash(patch *forgepb.Patch) error {
|
|||
|
||||
comment := cleanSubject(patch.Comment)
|
||||
|
||||
if patch.NewHash != "" {
|
||||
return nil
|
||||
}
|
||||
os.Chdir(repo.GetFullPath())
|
||||
newhash, err := findCommitBySubject(comment)
|
||||
if err != nil {
|
||||
return log.Errorf("patch: not found hash: %s %s %s %s %v", patch.CommitHash, patch.Namespace, comment, newhash, err)
|
||||
}
|
||||
if patch.NewHash == newhash {
|
||||
// patch was already set
|
||||
return nil
|
||||
}
|
||||
if patch.NewHash != "" {
|
||||
log.Infof("patch: hash MISMATCH %s old=%s new=%s name=%s\n", patch.Namespace, patch.NewHash, newhash, comment)
|
||||
return nil
|
||||
|
||||
patchId, err := repo.FindPatchId(newhash)
|
||||
if err != nil {
|
||||
return err
|
||||
}
|
||||
|
||||
patch.PatchId = patchId
|
||||
patch.NewHash = newhash
|
||||
|
||||
log.Info("patch: found hash:", patch.CommitHash, newhash, patch.Namespace, comment)
|
||||
return nil
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue