compiler/protogen: relax rules for valid import paths
The path "sub.example.com" is a valid Go import path and
should not be rejected. Relax the check to require at least one
dot or slash. Either way, it still prevents the situation where
a user erroneously treats this option as just the package name.
Change-Id: I3df811e1eae61c9d2a0b81c001a27cc7c08c3838
Reviewed-on: https://go-review.googlesource.com/c/protobuf/+/316949
Trust: Joe Tsai <joetsai@digital-static.net>
Trust: Joe Tsai <thebrokentoaster@gmail.com>
Reviewed-by: Damien Neil <dneil@google.com>
diff --git a/compiler/protogen/protogen_test.go b/compiler/protogen/protogen_test.go
index 613d6b9..4f5ceed 100644
--- a/compiler/protogen/protogen_test.go
+++ b/compiler/protogen/protogen_test.go
@@ -109,6 +109,14 @@
wantFilename: "golang.org/x/foo/filename",
},
{
+ desc: "go_package option sets import path without slashes",
+ goPackageOption: "golang.org;foo",
+ generate: true,
+ wantPackageName: "foo",
+ wantImportPath: "golang.org",
+ wantFilename: "golang.org/filename",
+ },
+ {
desc: "go_package option sets import path and package",
goPackageOption: "golang.org/x/foo;bar",
generate: true,