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

Unified Diff: vpython/application/subcommand_verify.go

Issue 2918623003: [vpython] Verify environment, named installs. (Closed)
Patch Set: Created 3 years, 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « vpython/application/subcommand_install.go ('k') | vpython/spec/spec.go » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: vpython/application/subcommand_verify.go
diff --git a/vpython/application/subcommand_verify.go b/vpython/application/subcommand_verify.go
index 95762377f1746947a984d2d5d17cac4709868db7..b9c3e43abe278c302a94f8fbe4fc85594c428e78 100644
--- a/vpython/application/subcommand_verify.go
+++ b/vpython/application/subcommand_verify.go
@@ -42,7 +42,10 @@ func (cr *verifyCommandRun) Run(app subcommands.Application, args []string, env
if err := a.opts.ResolveSpec(c); err != nil {
return errors.Annotate(err).Reason("failed to resolve specification").Err()
}
- if err := spec.Normalize(a.opts.EnvConfig.Spec, &a.opts.EnvConfig.Package); err != nil {
+ if a.opts.EnvConfig.Spec.Virtualenv == nil {
+ a.opts.EnvConfig.Spec.Virtualenv = &a.opts.EnvConfig.Package
+ }
+ if err := spec.NormalizeSpec(a.opts.EnvConfig.Spec); err != nil {
return errors.Annotate(err).Reason("failed to normalize specification").Err()
}
s := a.opts.EnvConfig.Spec
« no previous file with comments | « vpython/application/subcommand_install.go ('k') | vpython/spec/spec.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698