Skip to content

Commit 757f20f

Browse files
committed
app/appengine: delete yet more dead code
This is part of a series of CLs to clean up the build.golang.org App Engine app in prep for it to be modernized, refactored, and replaced, starting with deleting dead code. Some of this was missed in CL 208319. Updates golang/go#34744 Change-Id: I35adf4296b849dabed9d974e7eb58eeedc7d4293 Reviewed-on: https://go-review.googlesource.com/c/build/+/208397 Reviewed-by: Andrew Gerrand <[email protected]>
1 parent 1380f86 commit 757f20f

File tree

6 files changed

+5
-46
lines changed

6 files changed

+5
-46
lines changed

app/appengine/build.go

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -86,12 +86,10 @@ type Commit struct {
8686
ParentHash string
8787
Num int // Internal monotonic counter unique to this package.
8888

89-
User string
90-
Desc string `datastore:",noindex"`
91-
Time time.Time
92-
NeedsBenchmarking bool
93-
TryPatch bool
94-
Branch string
89+
User string
90+
Desc string `datastore:",noindex"`
91+
Time time.Time
92+
Branch string
9593

9694
// ResultData is the Data string of each build Result for this Commit.
9795
// For non-Go commits, only the Results for the current Go tip, weekly,

app/appengine/cron-part.yaml

Lines changed: 0 additions & 6 deletions
This file was deleted.

app/appengine/index.yaml

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -18,10 +18,3 @@ indexes:
1818
properties:
1919
- name: Time
2020
direction: desc
21-
22-
- kind: Commit
23-
ancestor: yes
24-
properties:
25-
- name: NeedsBenchmarking
26-
- name: Num
27-
direction: desc

app/appengine/init.go

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -41,19 +41,5 @@ func initHandler(w http.ResponseWriter, r *http.Request) {
4141
// Create secret key.
4242
key.Secret(c)
4343

44-
// Populate Go 1.4 tag. This is for bootstrapping the new feature of
45-
// building sub-repos against the stable release.
46-
// TODO(adg): remove this after Go 1.5 is released, at which point the
47-
// build system will pick up on the new release tag automatically.
48-
t := &Tag{
49-
Kind: "release",
50-
Name: "release-branch.go1.4",
51-
Hash: "883bc6ed0ea815293fe6309d66f967ea60630e87", // Go 1.4.2
52-
}
53-
if _, err := datastore.Put(c, t.Key(c), t); err != nil {
54-
logErr(w, r, err)
55-
return
56-
}
57-
5844
fmt.Fprint(w, "OK")
5945
}

app/appengine/static/style.css

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -269,11 +269,6 @@ tr.subheading2 th {
269269
padding: 2px 8px;
270270
}
271271

272-
.diff-benchmark {
273-
clear: both;
274-
padding-top: 5px;
275-
}
276-
277272
/* positioning elements */
278273

279274
.page {

app/appengine/ui.go

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -429,10 +429,6 @@ func (s builderOrder) Less(i, j int) bool {
429429
}
430430

431431
func builderPriority(builder string) (p int) {
432-
// Put -temp builders at the end, always.
433-
if strings.HasSuffix(builder, "-temp") {
434-
defer func() { p += 20 }()
435-
}
436432
// Group race builders together.
437433
if isRace(builder) {
438434
return 2
@@ -450,9 +446,6 @@ func isRace(s string) bool {
450446
}
451447

452448
func unsupported(builder string) bool {
453-
if strings.HasSuffix(builder, "-temp") {
454-
return true
455-
}
456449
return unsupportedOS(builderOS(builder))
457450
}
458451

@@ -688,7 +681,7 @@ func builderSpans(s []string) []builderSpan {
688681
for len(s) > 0 {
689682
i := 1
690683
os := builderOSOrRace(s[0])
691-
u := unsupportedOS(os) || strings.HasSuffix(s[0], "-temp")
684+
u := unsupportedOS(os)
692685
for i < len(s) && builderOSOrRace(s[i]) == os {
693686
i++
694687
}

0 commit comments

Comments
 (0)