Skip to content

P
protobuf
Commit 477f799f authored by kenton@google.com's avatar kenton@google.com
Browse files
Options

Improve error message regarding proto_path not matching .proto file.

parent 32f330f7
Hide whitespace changes
Inline Side-by-side
Showing with 10 additions and 2 deletions
src/google/protobuf/compiler/command_line_interface.cc
View file @ 477f799f
......@@ -439,7 +439,11 @@ bool CommandLineInterface::MakeInputsBeProtoPathRelative(
} else {
cerr << input_files_[i] << ": File does not reside within any path "
"specified using --proto_path (or -I). You must specify a "
"--proto_path which encompasses this file." << endl;
"--proto_path which encompasses this file. Note that the "
"proto_path must be an exact prefix of the .proto file "
"names -- protoc is too dumb to figure out when two paths "
"(e.g. absolute and relative) are equivalent (it's harder "
"than you think)." << endl;
}
return false;
}
......
src/google/protobuf/compiler/command_line_interface_unittest.cc
View file @ 477f799f
......@@ -882,7 +882,11 @@ TEST_F(CommandLineInterfaceTest, CwdRelativeInputNotMappedError) {
ExpectErrorText(
"$tmpdir/foo.proto: File does not reside within any path "
"specified using --proto_path (or -I). You must specify a "
"--proto_path which encompasses this file.\n");
"--proto_path which encompasses this file. Note that the "
"proto_path must be an exact prefix of the .proto file "
"names -- protoc is too dumb to figure out when two paths "
"(e.g. absolute and relative) are equivalent (it's harder "
"than you think).\n");
}
TEST_F(CommandLineInterfaceTest, CwdRelativeInputNotFoundAndNotMappedError) {
......
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