Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(158)

Side by Side Diff: deploytool/cmd/luci_deploy/path.go

Issue 2963503003: [errors] Greatly simplify common/errors package. (Closed)
Patch Set: fix nits Created 3 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « deploytool/cmd/luci_deploy/param.go ('k') | deploytool/cmd/luci_deploy/staging.go » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The LUCI Authors. All rights reserved. 1 // Copyright 2016 The LUCI Authors. All rights reserved.
2 // Use of this source code is governed under the Apache License, Version 2.0 2 // Use of this source code is governed under the Apache License, Version 2.0
3 // that can be found in the LICENSE file. 3 // that can be found in the LICENSE file.
4 4
5 package main 5 package main
6 6
7 import ( 7 import (
8 "io/ioutil" 8 "io/ioutil"
9 "os" 9 "os"
10 "path/filepath" 10 "path/filepath"
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
51 if filepath.Separator == '/' { 51 if filepath.Separator == '/' {
52 return path 52 return path
53 } 53 }
54 return strings.Replace(path, "/", string(filepath.Separator), -1) 54 return strings.Replace(path, "/", string(filepath.Separator), -1)
55 } 55 }
56 56
57 func withTempDir(f func(string) error) error { 57 func withTempDir(f func(string) error) error {
58 // Create a temporary directory. 58 // Create a temporary directory.
59 tdir, err := ioutil.TempDir("", "luci_deploytool") 59 tdir, err := ioutil.TempDir("", "luci_deploytool")
60 if err != nil { 60 if err != nil {
61 » » return errors.Annotate(err).Reason("failed to create tempdir").E rr() 61 » » return errors.Annotate(err, "failed to create tempdir").Err()
62 } 62 }
63 defer os.RemoveAll(tdir) 63 defer os.RemoveAll(tdir)
64 64
65 return f(tdir) 65 return f(tdir)
66 } 66 }
OLDNEW
« no previous file with comments | « deploytool/cmd/luci_deploy/param.go ('k') | deploytool/cmd/luci_deploy/staging.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698