Commit 9e93d501 authored by Brad Fitzpatrick's avatar Brad Fitzpatrick

api: add go1.1.txt; update cmd/api to use it

R=golang-dev, adg, r
CC=golang-dev
https://golang.org/cl/9250043
parent f322c786
This source diff could not be displayed because it is too large. You can view the blob instead.
This source diff could not be displayed because it is too large. You can view the blob instead.
...@@ -38,9 +38,7 @@ import ( ...@@ -38,9 +38,7 @@ import (
// Flags // Flags
var ( var (
// TODO(bradfitz): once Go 1.1 comes out, allow the -c flag to take a comma-separated checkFile = flag.String("c", "", "optional comma-separated filename(s) to check API against")
// list of files, rather than just one.
checkFile = flag.String("c", "", "optional filename to check API against")
allowNew = flag.Bool("allow_new", true, "allow API additions") allowNew = flag.Bool("allow_new", true, "allow API additions")
exceptFile = flag.String("except", "", "optional filename of packages that are allowed to change without triggering a failure in the tool") exceptFile = flag.String("except", "", "optional filename of packages that are allowed to change without triggering a failure in the tool")
nextFile = flag.String("next", "", "optional filename of tentative upcoming API features for the next release. This file can be lazily maintained. It only affects the delta warnings from the -c file printed on success.") nextFile = flag.String("next", "", "optional filename of tentative upcoming API features for the next release. This file can be lazily maintained. It only affects the delta warnings from the -c file printed on success.")
...@@ -186,7 +184,10 @@ func main() { ...@@ -186,7 +184,10 @@ func main() {
return return
} }
required := fileFeatures(*checkFile) var required []string
for _, file := range strings.Split(*checkFile, ",") {
required = append(required, fileFeatures(file)...)
}
optional := fileFeatures(*nextFile) optional := fileFeatures(*nextFile)
exception := fileFeatures(*exceptFile) exception := fileFeatures(*exceptFile)
fail = !compareAPI(bw, features, required, optional, exception) fail = !compareAPI(bw, features, required, optional, exception)
......
...@@ -150,7 +150,7 @@ time go run run.go ...@@ -150,7 +150,7 @@ time go run run.go
echo echo
echo '# Checking API compatibility.' echo '# Checking API compatibility.'
go tool api -c $GOROOT/api/go1.txt -next $GOROOT/api/next.txt -except $GOROOT/api/except.txt go tool api -c $GOROOT/api/go1.txt,$GOROOT/api/go1.1.txt -next $GOROOT/api/next.txt -except $GOROOT/api/except.txt
echo echo
echo ALL TESTS PASSED echo ALL TESTS PASSED
...@@ -121,7 +121,7 @@ set GOMAXPROCS=%OLDGOMAXPROCS% ...@@ -121,7 +121,7 @@ set GOMAXPROCS=%OLDGOMAXPROCS%
set OLDGOMAXPROCS= set OLDGOMAXPROCS=
echo # Checking API compatibility. echo # Checking API compatibility.
go tool api -c ..\api\go1.txt -next ..\api\next.txt -except ..\api\except.txt go tool api -c ..\api\go1.txt,..\api\go1.1.txt -next ..\api\next.txt -except ..\api\except.txt
if errorlevel 1 goto fail if errorlevel 1 goto fail
echo. echo.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment