Merge branch 'master' into source_cache_close

.github/ISSUE_TEMPLATE.md

@@ -5,7 +5,11 @@ everything here and write out the request, providing as much context as you can.
 
 -->
 
-### What version of Go (`go version`) and `dep` (`git describe --tags`) are you using?
+### What version of `dep` are you using (`dep version`)?
+<!--
+  If you installed `dep` via `go get`, report the version instead with
+  `cd $GOPATH/src/github.com/golang/dep && git describe --tags`
+-->
 
 ### What `dep` command did you run?
 
@@ -20,4 +24,3 @@ The output of `dep hash-inputs` may also be helpful to include.
 ### What did you expect to see?
 
 ### What did you see instead?
-

.travis.yml

@@ -1,4 +1,3 @@
----
 language: go
 sudo: false
 notifications:
@@ -6,6 +5,7 @@ notifications:
 jobs:
   include:
     - stage: test
+      go_import_path: github.com/golang/dep
       install:
         - go get -u honnef.co/go/tools/cmd/{gosimple,staticcheck}
         - npm install -g codeclimate-test-reporter
@@ -15,20 +15,16 @@ jobs:
       go: 1.8.x
       script:
         - go build -v ./cmd/dep
-        - PKGS=$(go list ./... | grep -v /vendor/)
-        - go vet $PKGS
-        - staticcheck $PKGS
-        - gosimple $PKGS
+        - ./hack/lint.bash
         - ./hack/validate-vendor.bash
-        - go build ./hack/licenseok
-        - find . -path ./vendor -prune -o -type f -name "*.go" -printf '%P\n' | xargs ./licenseok
-        - set -e; for pkg in $PKGS; do go test -race -coverprofile=profile.out -covermode=atomic $pkg; if [[ -f profile.out ]]; then cat profile.out >> coverage.txt; rm profile.out; fi; done
+        - ./hack/coverage.bash
       after_success:
         - codeclimate-test-reporter < coverage.txt
     # YAML alias, for settings shared across the simpler builds
     - &simple-test
-      go: 1.7.x
+      go: 1.9.x
       stage: test
+      go_import_path: github.com/golang/dep
       install: skip
       env:
         - DEPTESTBYPASS501=1
@@ -53,3 +49,26 @@ jobs:
         # Related: https://superuser.com/questions/1044130/why-am-i-having-how-can-i-fix-this-error-shell-session-update-command-not-f
         - trap EXIT
         - go test -race $(go list ./... | grep -v vendor)
+    - go: 1.8.x
+      stage: deploy
+      go_import_path: github.com/golang/dep
+      install: skip
+      script: skip
+      before_deploy:
+        - ./hack/build-all.bash
+      deploy:
+        - provider: releases
+          api_key:
+            secure: fL9GX11J3JLizEBTPZHN32wuAT91eAJsGl0kjlAdIc6Lb/9UCe1XZGgFnpQFN4qo/S+omhHBDbM6Ty1xhNy7xmjDecpQGDU8Rmap9Oll0TuxqMigG+njOuPp5VUYPofPP0PGKdxAcYg+KaFM7x0o2rK+qA046NHwo2gH1BbE+bn55TZglEajEfc8j9iX4jt96KC7zlu+WiKArLmfUtlrI8m8ZYgbYcvFmlYjeCiEqlNhvNL59ejug9Rl0PLtPbamqVXkGLafYtekgPCb4WSxBiCt8pq5Rb5svk9YcdXpiaWQhZjMPAuKN6BrmN2lw1PiXzADUG5fjvNc8eo2HY70GD2utU9cAsY8VIafhoH5n6uM1WI8MHwDfd7P1PiQA3ZGQ8CPwk4q/8HSfQU9ap7vZgSF63pTIbtlviyIG67orOJE9PWWncl9olYM946UylZu6m3hWI/rmJxOeJ1UJjym/3GNPMRfKubaGhV/TyRdM0bKX4M0cXHU6k/ESVFupGXdKRt4RpvkD4/1Km6b2OShW6PNI+ifFspnJr7obkI7dm7ubySdnNz4lMv9WWymxRpMVc8hUAhuoDvXeZJq7pSnkjBEWDxIRoTkA93CU3/Rf7MFYCJMnGSqjcxWUpIfCAk2/r4BqL9NQnqBvvVt+MYi64QaD5n7ZF3dVbr6HZ2zjSU=
+          file:
+            - release/dep-linux-amd64
+            - release/dep-linux-amd64.sha256
+            - release/dep-darwin-amd64
+            - release/dep-darwin-amd64.sha256
+            - release/dep-windows-amd64
+            - release/dep-windows-amd64.sha256
+          skip_cleanup: true
+          on:
+            repo: golang/dep
+            branch: master
+            tags: true

README.md

@@ -8,7 +8,7 @@
 
 ## Dep
 
-`dep` is a prototype dependency management tool for Go. It requires Go 1.7 or newer to compile.
+`dep` is a prototype dependency management tool for Go. It requires Go 1.8 or newer to compile.
 
 `dep` is the official _experiment_, but not yet the official tool. Check out the [Roadmap](https://github.com/golang/dep/wiki/Roadmap) for more on what this means!
 
@@ -39,11 +39,7 @@ That said, keep in mind the following:
 
 ## Setup
 
-Get the tool via
-
-```sh
-$ go get -u github.com/golang/dep/cmd/dep
-```
+Grab the latest binary from the [releases](https://github.com/golang/dep/releases) page.
 
 On macOS you can install or upgrade to the latest released version with Homebrew:
 
@@ -52,6 +48,12 @@ $ brew install dep
 $ brew upgrade dep
 ```
 
+If you're interested in hacking on `dep`, you can install via `go get`:
+
+```sh
+go get -u github.com/golang/dep/cmd/dep
+```
+
 To start managing dependencies using dep, run the following from your project's root directory:
 
 ```sh

cmd/dep/main.go

@@ -62,6 +62,7 @@ func (c *Config) Run() (exitCode int) {
 		&ensureCommand{},
 		&hashinCommand{},
 		&pruneCommand{},
+		&versionCommand{},
 	}
 
 	examples := [][2]string{

cmd/dep/status.go

@@ -349,7 +349,9 @@ func runStatusAll(ctx *dep.Ctx, out outputter, p *dep.Project, sm gps.SourceMana
 	// deterministically ordered. (This may be superfluous if the lock is always
 	// written in alpha order, but it doesn't hurt to double down.)
 	slp := p.Lock.Projects()
-	sort.Sort(dep.SortedLockedProjects(slp))
+	sort.Slice(slp, func(i, j int) bool {
+		return slp[i].Ident().Less(slp[j].Ident())
+	})
 
 	if bytes.Equal(s.HashInputs(), p.Lock.SolveMeta.InputsDigest) {
 		// If these are equal, we're guaranteed that the lock is a transitively

cmd/dep/version.go

@@ -0,0 +1,45 @@
+// Copyright 2016 The Go Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
+package main
+
+import (
+	"flag"
+	"runtime"
+
+	"github.com/golang/dep"
+)
+
+var (
+	version    = "devel"
+	buildDate  string
+	commitHash string
+)
+
+const versionHelp = `Show the dep version information`
+
+func (cmd *versionCommand) Name() string { return "version" }
+func (cmd *versionCommand) Args() string {
+	return ""
+}
+func (cmd *versionCommand) ShortHelp() string { return versionHelp }
+func (cmd *versionCommand) LongHelp() string  { return versionHelp }
+func (cmd *versionCommand) Hidden() bool      { return false }
+
+func (cmd *versionCommand) Register(fs *flag.FlagSet) {}
+
+type versionCommand struct{}
+
+func (cmd *versionCommand) Run(ctx *dep.Ctx, args []string) error {
+	ctx.Out.Printf(`dep:
+ version     : %s
+ build date  : %s
+ git hash    : %s
+ go version  : %s
+ go compiler : %s
+ platform    : %s/%s
+`, version, buildDate, commitHash,
+		runtime.Version(), runtime.Compiler, runtime.GOOS, runtime.GOARCH)
+	return nil
+}

hack/build-all.bash

@@ -0,0 +1,37 @@
+#!/usr/bin/env bash
+# Copyright 2017 The Go Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style
+# license that can be found in the LICENSE file.
+#
+# This script will build dep and calculate hash for each
+# (DEP_BUILD_PLATFORMS, DEP_BUILD_ARCHS) pair.
+# DEP_BUILD_PLATFORMS="linux" DEP_BUILD_ARCHS="amd64" ./hack/build-all.sh
+# can be called to build only for linux-amd64
+
+set -e
+
+VERSION=$(git describe --tags --dirty)
+COMMIT_HASH=$(git rev-parse --short HEAD 2>/dev/null)
+DATE=$(date --iso-8601)
+
+GO_BUILD_CMD="go build -a -installsuffix cgo"
+GO_BUILD_LDFLAGS="-s -w -X main.commitHash=$COMMIT_HASH -X main.buildDate=$DATE -X main.version=$VERSION"
+
+if [ -z "$DEP_BUILD_PLATFORMS" ]; then
+    DEP_BUILD_PLATFORMS="linux windows darwin"
+fi
+
+if [ -z "$DEP_BUILD_ARCHS" ]; then
+    DEP_BUILD_ARCHS="amd64"
+fi
+
+mkdir -p release
+
+for OS in ${DEP_BUILD_PLATFORMS[@]}; do
+  for ARCH in ${DEP_BUILD_ARCHS[@]}; do
+    echo "Building for $OS/$ARCH"
+    GOARCH=$ARCH GOOS=$OS CGO_ENABLED=0 $GO_BUILD_CMD -ldflags "$GO_BUILD_LDFLAGS"\
+     -o "release/dep-$OS-$ARCH" ./cmd/dep/
+    sha256sum "release/dep-$OS-$ARCH" > "release/dep-$OS-$ARCH".sha256
+  done
+done

hack/coverage.bash

@@ -0,0 +1,16 @@
+#!/usr/bin/env bash
+# Copyright 2017 The Go Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style
+# license that can be found in the LICENSE file.
+#
+# This script will generate coverage.txt
+set -e
+
+PKGS=$(go list ./... | grep -v /vendor/)
+for pkg in $PKGS; do
+  go test -race -coverprofile=profile.out -covermode=atomic $pkg
+  if [[ -f profile.out ]]; then
+    cat profile.out >> coverage.txt
+    rm profile.out
+  fi
+done

hack/lint.bash

@@ -0,0 +1,12 @@
+#!/usr/bin/env bash
+# Copyright 2017 The Go Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style
+# license that can be found in the LICENSE file.
+#
+# This script will validate code with various linters
+set -e
+
+PKGS=$(go list ./... | grep -v /vendor/)
+go vet $PKGS
+staticcheck $PKGS
+gosimple $PKGS

hack/validate-licence.bash

@@ -0,0 +1,12 @@
+#!/usr/bin/env bash
+# Copyright 2017 The Go Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style
+# license that can be found in the LICENSE file.
+#
+# This script will build licenseok and run it on all
+# source files to check licence
+set -e
+
+go build ./hack/licenseok
+find . -path ./vendor -prune -o -type f -name "*.go"\
+ -printf '%P\n' | xargs ./licenseok

internal/fs/fs.go

@@ -109,7 +109,7 @@ func RenameWithFallback(src, dst string) error {
 		return errors.Wrapf(err, "cannot stat %s", src)
 	}
 
-	err = rename(src, dst)
+	err = os.Rename(src, dst)
 	if err == nil {
 		return nil
 	}

internal/fs/rename.go

@@ -3,7 +3,6 @@
 // license that can be found in the LICENSE file.
 
 // +build !windows
-// +build go1.8
 
 package fs
 
@@ -14,10 +13,6 @@ import (
 	"github.com/pkg/errors"
 )
 
-func rename(src, dst string) error {
-	return os.Rename(src, dst)
-}
-
 // renameFallback attempts to determine the appropriate fallback to failed rename
 // operation depending on the resulting error.
 func renameFallback(err error, src, dst string) error {

internal/fs/rename_go17.go

@@ -1,50 +0,0 @@
-// Copyright 2016 The Go Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style
-// license that can be found in the LICENSE file.
-
-// +build !windows
-// +build !go1.8
-
-package fs
-
-import (
-	"os"
-	"syscall"
-
-	"github.com/pkg/errors"
-)
-
-func rename(src, dst string) error {
-	fi, err := os.Stat(src)
-	if err != nil {
-		return errors.Wrapf(err, "cannot stat %s", src)
-	}
-
-	// In go 1.8, the behavior of os.Rename changed on non-Windows platforms. It no
-	// longer allows renames that would replace an existing directory. This has
-	// always been the case on Windows, though.
-	//
-	// For consistency, we replicate the go 1.8 behavior in earlier go versions here.
-	if dstfi, err := os.Stat(dst); fi.IsDir() && err == nil && dstfi.IsDir() {
-		return errors.Errorf("cannot rename directory %s to existing dst %s", src, dst)
-	}
-
-	return os.Rename(src, dst)
-}
-
-// renameFallback attempts to determine the appropriate fallback to failed rename
-// operation depending on the resulting error.
-func renameFallback(err error, src, dst string) error {
-	// Rename may fail if src and dst are on different devices; fall back to
-	// copy if we detect that case. syscall.EXDEV is the common name for the
-	// cross device link error which has varying output text across different
-	// operating systems.
-	terr, ok := err.(*os.LinkError)
-	if !ok {
-		return err
-	} else if terr.Err != syscall.EXDEV {
-		return errors.Wrapf(terr, "link error: cannot rename %s to %s", src, dst)
-	}
-
-	return renameByCopy(src, dst)
-}

internal/fs/rename_windows.go

@@ -13,10 +13,6 @@ import (
 	"github.com/pkg/errors"
 )
 
-func rename(src, dst string) error {
-	return os.Rename(src, dst)
-}
-
 // renameFallback attempts to determine the appropriate fallback to failed rename
 // operation depending on the resulting error.
 func renameFallback(err error, src, dst string) error {

internal/gps/constraints.go

@@ -331,7 +331,9 @@ func (m ProjectConstraints) asSortedSlice() []ProjectConstraint {
 		k++
 	}
 
-	sort.Stable(sortedConstraints(pcs))
+	sort.SliceStable(pcs, func(i, j int) bool {
+		return pcs[i].Ident.Less(pcs[j].Ident)
+	})
 	return pcs
 }
 
@@ -348,7 +350,9 @@ func (m ProjectConstraints) overrideAll(pcm ProjectConstraints) (out []workingCo
 		k++
 	}
 
-	sort.Stable(sortedWC(out))
+	sort.SliceStable(out, func(i, j int) bool {
+		return out[i].Ident.Less(out[j].Ident)
+	})
 	return
 }
 
@@ -389,15 +393,3 @@ func (m ProjectConstraints) override(pr ProjectRoot, pp ProjectProperties) worki
 
 	return wc
 }
-
-type sortedConstraints []ProjectConstraint
-
-func (s sortedConstraints) Len() int           { return len(s) }
-func (s sortedConstraints) Swap(i, j int)      { s[i], s[j] = s[j], s[i] }
-func (s sortedConstraints) Less(i, j int) bool { return s[i].Ident.less(s[j].Ident) }
-
-type sortedWC []workingConstraint
-
-func (s sortedWC) Len() int           { return len(s) }
-func (s sortedWC) Swap(i, j int)      { s[i], s[j] = s[j], s[i] }
-func (s sortedWC) Less(i, j int) bool { return s[i].Ident.less(s[j].Ident) }

internal/gps/identifier.go

@@ -80,14 +80,14 @@ type ProjectIdentifier struct {
 	Source      string
 }
 
-func (i ProjectIdentifier) less(j ProjectIdentifier) bool {
+// Less compares by ProjectRoot then normalized Source.
+func (i ProjectIdentifier) Less(j ProjectIdentifier) bool {
 	if i.ProjectRoot < j.ProjectRoot {
 		return true
 	}
 	if j.ProjectRoot < i.ProjectRoot {
 		return false
 	}
-
 	return i.normalizedSource() < j.normalizedSource()
 }
 

internal/gps/lock.go

@@ -39,20 +39,8 @@ func LocksAreEq(l1, l2 Lock, checkHash bool) bool {
 		return false
 	}
 
-	// Check if the slices are sorted already. If they are, we can compare
-	// without copying. Otherwise, we have to copy to avoid altering the
-	// original input.
-	sp1, sp2 := lpsorter(p1), lpsorter(p2)
-	if len(p1) > 1 && !sort.IsSorted(sp1) {
-		p1 = make([]LockedProject, len(p1))
-		copy(p1, l1.Projects())
-		sort.Sort(lpsorter(p1))
-	}
-	if len(p2) > 1 && !sort.IsSorted(sp2) {
-		p2 = make([]LockedProject, len(p2))
-		copy(p2, l2.Projects())
-		sort.Sort(lpsorter(p2))
-	}
+	p1 = sortedLockedProjects(p1)
+	p2 = sortedLockedProjects(p2)
 
 	for k, lp := range p1 {
 		if !lp.Eq(p2[k]) {
@@ -62,6 +50,21 @@ func LocksAreEq(l1, l2 Lock, checkHash bool) bool {
 	return true
 }
 
+// sortedLockedProjects returns a sorted copy of lps, or itself if already sorted.
+func sortedLockedProjects(lps []LockedProject) []LockedProject {
+	if len(lps) <= 1 || sort.SliceIsSorted(lps, func(i, j int) bool {
+		return lps[i].Ident().Less(lps[j].Ident())
+	}) {
+		return lps
+	}
+	cp := make([]LockedProject, len(lps))
+	copy(cp, lps)
+	sort.Slice(cp, func(i, j int) bool {
+		return cp[i].Ident().Less(cp[j].Ident())
+	})
+	return cp
+}
+
 // LockedProject is a single project entry from a lock file. It expresses the
 // project's name, one or both of version and underlying revision, the network
 // URI for accessing it, the path at which it should be placed within a vendor
@@ -230,23 +233,3 @@ func prepLock(l Lock) safeLock {
 
 	return rl
 }
-
-// SortLockedProjects sorts a slice of LockedProject in alphabetical order by
-// ProjectIdentifier.
-func SortLockedProjects(lps []LockedProject) {
-	sort.Stable(lpsorter(lps))
-}
-
-type lpsorter []LockedProject
-
-func (lps lpsorter) Swap(i, j int) {
-	lps[i], lps[j] = lps[j], lps[i]
-}
-
-func (lps lpsorter) Len() int {
-	return len(lps)
-}
-
-func (lps lpsorter) Less(i, j int) bool {
-	return lps[i].Ident().less(lps[j].Ident())
-}

internal/gps/lock_test.go

@@ -6,6 +6,7 @@ package gps
 
 import (
 	"reflect"
+	"sort"
 	"testing"
 )
 
@@ -20,7 +21,9 @@ func TestLockedProjectSorting(t *testing.T) {
 	lps2 := make([]LockedProject, len(lps))
 	copy(lps2, lps)
 
-	SortLockedProjects(lps2)
+	sort.SliceStable(lps2, func(i, j int) bool {
+		return lps2[i].Ident().Less(lps2[j].Ident())
+	})
 
 	// only the two should have switched positions
 	lps[0], lps[2] = lps[2], lps[0]

internal/gps/lockdiff.go

@@ -74,20 +74,8 @@ func DiffLocks(l1 Lock, l2 Lock) *LockDiff {
 
 	p1, p2 := l1.Projects(), l2.Projects()
 
-	// Check if the slices are sorted already. If they are, we can compare
-	// without copying. Otherwise, we have to copy to avoid altering the
-	// original input.
-	sp1, sp2 := lpsorter(p1), lpsorter(p2)
-	if len(p1) > 1 && !sort.IsSorted(sp1) {
-		p1 = make([]LockedProject, len(p1))
-		copy(p1, l1.Projects())
-		sort.Sort(lpsorter(p1))
-	}
-	if len(p2) > 1 && !sort.IsSorted(sp2) {
-		p2 = make([]LockedProject, len(p2))
-		copy(p2, l2.Projects())
-		sort.Sort(lpsorter(p2))
-	}
+	p1 = sortedLockedProjects(p1)
+	p2 = sortedLockedProjects(p2)
 
 	diff := LockDiff{}
 

internal/gps/manager_test.go

@@ -53,7 +53,7 @@ func mkNaiveSM(t *testing.T) (*SourceMgr, func()) {
 
 	return sm, func() {
 		sm.Release()
-		err := removeAll(cpath)
+		err := os.RemoveAll(cpath)
 		if err != nil {
 			t.Errorf("removeAll failed: %s", err)
 		}
@@ -74,7 +74,7 @@ func remakeNaiveSM(osm *SourceMgr, t *testing.T) (*SourceMgr, func()) {
 
 	return sm, func() {
 		sm.Release()
-		err := removeAll(cpath)
+		err := os.RemoveAll(cpath)
 		if err != nil {
 			t.Errorf("removeAll failed: %s", err)
 		}
@@ -109,7 +109,7 @@ func TestSourceManagerInit(t *testing.T) {
 	}
 
 	sm.Release()
-	err = removeAll(cpath)
+	err = os.RemoveAll(cpath)
 	if err != nil {
 		t.Errorf("removeAll failed: %s", err)
 	}
@@ -125,7 +125,7 @@ func TestSourceManagerInit(t *testing.T) {
 	}
 
 	sm.Release()
-	err = removeAll(cpath)
+	err = os.RemoveAll(cpath)
 	if err != nil {
 		t.Errorf("removeAll failed: %s", err)
 	}
@@ -152,7 +152,7 @@ func TestSourceInit(t *testing.T) {
 
 	defer func() {
 		sm.Release()
-		err := removeAll(cpath)
+		err := os.RemoveAll(cpath)
 		if err != nil {
 			t.Errorf("removeAll failed: %s", err)
 		}

internal/gps/remove_go16.go

@@ -1,48 +0,0 @@
-// Copyright 2017 The Go Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style
-// license that can be found in the LICENSE file.
-
-// +build !go1.7
-
-package gps
-
-import (
-	"os"
-	"path/filepath"
-	"runtime"
-)
-
-// removeAll removes path and any children it contains. It deals correctly with
-// removal on Windows where, prior to Go 1.7, there were issues when files were
-// set to read-only.
-func removeAll(path string) error {
-	// Only need special handling for windows
-	if runtime.GOOS != "windows" {
-		return os.RemoveAll(path)
-	}
-
-	// Simple case: if Remove works, we're done.
-	err := os.Remove(path)
-	if err == nil || os.IsNotExist(err) {
-		return nil
-	}
-
-	// make sure all files are writable so we can delete them
-	err = filepath.Walk(path, func(path string, info os.FileInfo, err error) error {
-		if err != nil && err != filepath.SkipDir {
-			// walk gave us some error, give it back.
-			return err
-		}
-		mode := info.Mode()
-		if mode|0200 == mode {
-			return nil
-		}
-
-		return os.Chmod(path, mode|0200)
-	})
-	if err != nil {
-		return err
-	}
-
-	return os.Remove(path)
-}

internal/gps/remove_go17.go

@@ -1,15 +0,0 @@
-// Copyright 2017 The Go Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style
-// license that can be found in the LICENSE file.
-
-// +build go1.7
-
-package gps
-
-import "os"
-
-// go1.7 and later deal with the file perms issue in os.RemoveAll(), so our
-// workaround is no longer necessary.
-func removeAll(path string) error {
-	return os.RemoveAll(path)
-}

internal/gps/result.go

@@ -98,7 +98,7 @@ func WriteDepTree(basedir string, l Lock, sm SourceManager, sv bool, logger *log
 			logger.Println(" * ", err)
 		}
 
-		removeAll(basedir)
+		os.RemoveAll(basedir)
 
 		return errors.New("failed to write dep tree")
 	}

internal/gps/solver.go

@@ -1123,7 +1123,7 @@ func (s *solver) unselectedComparator(i, j int) bool {
 	case !ilock && jlock:
 		return false
 	case ilock && jlock:
-		return iname.less(jname)
+		return iname.Less(jname)
 	}
 
 	// Now, sort by number of available versions. This will trigger network
@@ -1152,7 +1152,7 @@ func (s *solver) unselectedComparator(i, j int) bool {
 	}
 
 	// Finally, if all else fails, fall back to comparing by name
-	return iname.less(jname)
+	return iname.Less(jname)
 }
 
 func (s *solver) fail(id ProjectIdentifier) {

internal/gps/source.go

@@ -376,7 +376,7 @@ func (sg *sourceGateway) listPackages(ctx context.Context, pr ProjectRoot, v Ver
 			return pkgtree.PackageTree{}, err
 		}
 
-		err = sg.suprvsr.do(ctx, label, ctGetManifestAndLock, func(ctx context.Context) error {
+		err = sg.suprvsr.do(ctx, label, ctListPackages, func(ctx context.Context) error {
 			ptree, err = sg.src.listPackages(ctx, pr, r)
 			return err
 		})
@@ -417,7 +417,7 @@ func (sg *sourceGateway) convertToRevision(ctx context.Context, v Version) (Revi
 
 	// The version list is out of date; it's possible this version might
 	// show up after loading it.
-	_, err := sg.require(ctx, sourceIsSetUp|sourceHasLatestVersionList|sourceHasLatestLocally)
+	_, err := sg.require(ctx, sourceIsSetUp|sourceHasLatestVersionList)
 	if err != nil {
 		return "", err
 	}

internal/gps/source_test.go

@@ -9,6 +9,7 @@ import (
 	"fmt"
 	"io/ioutil"
 	"log"
+	"os"
 	"reflect"
 	"testing"
 
@@ -32,7 +33,7 @@ func testSourceGateway(t *testing.T) {
 	bgc := context.Background()
 	ctx, cancelFunc := context.WithCancel(bgc)
 	defer func() {
-		removeAll(cachedir)
+		os.RemoveAll(cachedir)
 		cancelFunc()
 	}()
 

internal/gps/strip_vendor.go

@@ -32,7 +32,7 @@ func stripVendor(path string, info os.FileInfo, err error) error {
 		}
 
 		if info.IsDir() {
-			if err := removeAll(path); err != nil {
+			if err := os.RemoveAll(path); err != nil {
 				return err
 			}
 			return filepath.SkipDir

internal/gps/strip_vendor_windows.go

@@ -40,7 +40,7 @@ func stripVendor(path string, info os.FileInfo, err error) error {
 				}
 
 			case dir:
-				if err := removeAll(path); err != nil {
+				if err := os.RemoveAll(path); err != nil {
 					return err
 				}
 				return filepath.SkipDir

internal/gps/vcs_source_test.go

@@ -49,7 +49,7 @@ func testGitSourceInteractions(t *testing.T) {
 		t.Errorf("Failed to create temp dir: %s", err)
 	}
 	defer func() {
-		if err := removeAll(cpath); err != nil {
+		if err := os.RemoveAll(cpath); err != nil {
 			t.Errorf("removeAll failed: %s", err)
 		}
 	}()
@@ -145,7 +145,7 @@ func testGopkginSourceInteractions(t *testing.T) {
 		t.Errorf("Failed to create temp dir: %s", err)
 	}
 	defer func() {
-		if err := removeAll(cpath); err != nil {
+		if err := os.RemoveAll(cpath); err != nil {
 			t.Errorf("removeAll failed: %s", err)
 		}
 	}()
@@ -296,7 +296,7 @@ func testBzrSourceInteractions(t *testing.T) {
 		t.Errorf("Failed to create temp dir: %s", err)
 	}
 	defer func() {
-		if err := removeAll(cpath); err != nil {
+		if err := os.RemoveAll(cpath); err != nil {
 			t.Errorf("removeAll failed: %s", err)
 		}
 	}()
@@ -406,7 +406,7 @@ func testHgSourceInteractions(t *testing.T) {
 		t.Errorf("Failed to create temp dir: %s", err)
 	}
 	defer func() {
-		if err := removeAll(cpath); err != nil {
+		if err := os.RemoveAll(cpath); err != nil {
 			t.Errorf("removeAll failed: %s", err)
 		}
 	}()

lock.go

@@ -149,7 +149,9 @@ func (l *Lock) toRaw() rawLock {
 		Projects: make([]rawLockedProject, len(l.P)),
 	}
 
-	sort.Sort(SortedLockedProjects(l.P))
+	sort.Slice(l.P, func(i, j int) bool {
+		return l.P[i].Ident().Less(l.P[j].Ident())
+	})
 
 	for k, lp := range l.P {
 		id := lp.Ident()
@@ -197,21 +199,3 @@ func LockFromSolution(in gps.Solution) *Lock {
 	copy(l.P, p)
 	return l
 }
-
-// SortedLockedProjects implements sort.Interface.
-type SortedLockedProjects []gps.LockedProject
-
-func (s SortedLockedProjects) Len() int      { return len(s) }
-func (s SortedLockedProjects) Swap(i, j int) { s[i], s[j] = s[j], s[i] }
-func (s SortedLockedProjects) Less(i, j int) bool {
-	l, r := s[i].Ident(), s[j].Ident()
-
-	if l.ProjectRoot < r.ProjectRoot {
-		return true
-	}
-	if r.ProjectRoot < l.ProjectRoot {
-		return false
-	}
-
-	return l.Source < r.Source
-}