Skip to content

Commit

Permalink
provenance: fix the order of the build steps
Browse files Browse the repository at this point in the history
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
  • Loading branch information
tonistiigi committed Dec 20, 2022
1 parent 103a0a1 commit ad85ce7
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions solver/llbsolver/provenance/buildconfig.go
Original file line number Diff line number Diff line change
Expand Up @@ -141,11 +141,6 @@ func toBuildSteps(def *pb.Definition) ([]BuildStep, map[digest.Digest]int, error
if err != nil {
return nil, nil, err
}
for i := 0; i < len(dgsts)/2; i++ {
j := len(dgsts) - 1 - i
dgsts[i], dgsts[j] = dgsts[j], dgsts[i]
}

indexes := map[digest.Digest]int{}
for i, dgst := range dgsts {
indexes[dgst] = i
Expand Down Expand Up @@ -179,7 +174,6 @@ func walkDigests(dgsts []digest.Digest, ops map[digest.Digest]*pb.Op, dgst diges
if op == nil {
return nil, errors.Errorf("invalid nil input %v", dgst)
}
dgsts = append(dgsts, dgst)
visited[dgst] = struct{}{}
for _, inp := range op.Inputs {
var err error
Expand All @@ -188,5 +182,6 @@ func walkDigests(dgsts []digest.Digest, ops map[digest.Digest]*pb.Op, dgst diges
return nil, err
}
}
dgsts = append(dgsts, dgst)
return dgsts, nil
}

0 comments on commit ad85ce7

Please sign in to comment.