-
Notifications
You must be signed in to change notification settings - Fork 539
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
生成 model 文件路径不符合预期 #1154
Comments
我在 #1155 试着修复了一下这个问题,不清楚是否是这个功能设计时的本意,请 review。 |
你好, 这是我使用的 proto 文件 syntax = "proto3";
package hello;
option go_package = "hertz/hello";
import "api.proto";
message HelloReq {
string Name = 1[(api.query)="name"];
}
message HelloResp {
string RespBody = 1;
}
service HelloService {
rpc Method1(HelloReq) returns(HelloResp) {
option (api.get) = "/hello";
}
} 依据 go_package 定义, 它应该生成在 biz/model/hello/hello.pb.go 或者你可以提供你的 proto 用于我们来确定问题的来源 |
option go_package = "hertz/hello"; 不应该是 biz/model/hertz/hello/hello.pb.go 吗?实际上它的生成文件是在 biz/model/hello.pb.go |
ok, 确实是存在相应的问题的, 我来 cr 一下 |
但是 预期生成路径应该是 biz/model + go_package 中定义的路径, 所以说 biz/model/hello/hello.pb.go 这个路径是没有错误的 |
这块最好的选择是重新命名 module,例如 github.com/cloudwego/hello 这样。 我看看是不是也把同学的优化合进去 |
Describe the bug
我在 proto 文件中定义了 go_packages,然后我使用 hz update 去生成相应的model文件,结果生成的文件路径与我在 proto 定义的文件路径不符
Replay
我的项目名为 hello
hz new -module hello -I idl -idl idl/hello/hello.proto
hz update -I idl -idl idl/hello/hello.proto
Note
只要我的go_packages 路径中包含或者等于 hello,这个实际生成的 pb 文件路径就会有问题
Hertz version:
The text was updated successfully, but these errors were encountered: