由.proto文件生成pojo对引用嵌套enum支持有问题 about jprotobuf

[xiemalin]

这个的确有问题，临时解决方案，把
enum DataVisibility {
PUBLIC = 1;
PRIVATE = 2;
}定义成全局
修改DataStatus定义如下
message DataStatus {
optional DataVisibility visibility = 1;
}

[abop]

首先感谢下这个项目，希望能尽快解决这个feature。
最近项目比较紧，先work around了，两周后估计有时间我也会看看能不能提个pr，你这边进度也在评论里同步一下吧。
再次感谢！

[xiemalin]

修复版本已发布， 版本号为 1.9.5

[abop]

感谢🙏
matthew xie [email protected]于2016年2月25日 周四20:50写道：

修复版本已发布， 版本号为 1.9.5

—
Reply to this email directly or view it on GitHub
#40 (comment).

[abop]

jprotobuf版本：1.9.5/1.9.6

如果proto文件有package声明，仍会触发之前的异常

src/test/java/com/baidu/bjf/remoting/protobuf/idlproxy/ProtobufIDLProxyTest.java

        StringBuilder idl = new StringBuilder();
        // 这里增加package声明
        idl.append("package mypkg;\n");
        idl.append("message DataInfo {\n");
        idl.append("   enum DataVisibility {\n");

        idl.append("   PUBLIC = 1;\n");
        idl.append("   PRIVATE = 2;\n");
        idl.append("   }");
        idl.append("    optional DataVisibility v = 1;\n");
        idl.append("   message SubDataInfo {\n");
        idl.append("      optional string name =1;\n");
        idl.append("      message Sub2DataInfo {\n");
        idl.append("        required int32 age = 1;\n");
        idl.append("      }\n");
        idl.append("   }\n");

        idl.append("}\n");


        idl.append("message DataStatus {\n");
        idl.append("   optional DataInfo.DataVisibility visibility = 1; \n");
        idl.append("   optional DataInfo.SubDataInfo subDataInfo = 2;\n");
        idl.append("   optional DataInfo.SubDataInfo.Sub2DataInfo sub2DataInfo = 3;\n");
        idl.append("}");

        System.out.println(idl.toString());
        ProtobufIDLProxy.generateSource(idl.toString(), new File("/"));

[abop]

需要提新的issue么？

[xiemalin]

我重新打开这个issue吧

[xiemalin]

1.9.7-SNAPSHOT 先使用这个版本进行测试，目前还需要更多场景进行验证

[abop]

1.9.7-SNAPSHOT 没上传maven仓库么？

[xiemalin]

前天就已上传了

[abop]

http://repo1.maven.org/maven2/com/baidu/jprotobuf/ 真没有

[xiemalin]

https://oss.sonatype.org/content/repositories/snapshots/com/baidu/jprotobuf/1.9.7-SNAPSHOT/

[abop]

不好意思 应该是我们公司镜像同步的问题，我先下载下来测试

[abop]

1.9.7-SNAPSHOT版本 import语句会导致抛错。

用例：
file1: Pk1.proto

package mypkg1;
message MyMsg {
    optional string msg = 1;
}

file2: Pk2.Proto

package mypkg2;
import Pk1.proto;
message MyMsg2 {
    optional mypkg1.MyMsg msg = 1;
}

运行：

try {
    ProtobufIDLProxy.generateSource(new File("Pk2.proto"), new File("gen/"));
} catch (IOException e) {
    e.printStackTrace();
}

抛异常：

Exception in thread "main" java.lang.RuntimeException: Message 'MyMsg2' depend on message 'mypkg1.MyMsg' is missed
    at com.baidu.bjf.remoting.protobuf.ProtobufIDLProxy.hasDependency(ProtobufIDLProxy.java:529)
    at com.baidu.bjf.remoting.protobuf.ProtobufIDLProxy.createClass(ProtobufIDLProxy.java:433)
...

[xiemalin]

这个场景的确没有测试过，需要解决

[xiemalin]

1.9.7-SNAPSHOT 已更新，修正 import方式下的引入问题

[xiemalin]

基本的测试已经通过， 1.9.7版本已发布