languages/go
languages/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-09-04 23:44:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

misc/dashboard/app: fix build failure notifications

Browse source 
The datastore.Query methods once mutated the Query value, but now they return
a derivative query, so the Hash= and ParentHash= filters were not being
applied.

R=golang-dev, bradfitz, dsymonds
CC=golang-dev
https://golang.org/cl/6300058
This commit is contained in:
Andrew Gerrand 2012-06-07 09:27:39 +10:00
parent 42a76efc92
commit 5528211158
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
misc/dashboard/app/build/notify.go
View file

@ -45,7 +45,7 @@ func notifyOnFailure(c appengine.Context, com *Commit, builder string) error {
if cr.OK {
// This commit is OK. Notify if next Commit is broken.
next := new(Commit)
q.Filter("ParentHash=", com.Hash)
q = q.Filter("ParentHash=", com.Hash)
if err := firstMatch(c, q, next); err != nil {
if err == datastore.ErrNoSuchEntity {
// OK at tip, no notification necessary.
@ -61,7 +61,7 @@ func notifyOnFailure(c appengine.Context, com *Commit, builder string) error {
} else {
// This commit is broken. Notify if the previous Commit is OK.
prev := new(Commit)
q.Filter("Hash=", com.ParentHash)
q = q.Filter("Hash=", com.ParentHash)
if err := firstMatch(c, q, prev); err != nil {
if err == datastore.ErrNoSuchEntity {
// No previous result, let the backfill of