-
Notifications
You must be signed in to change notification settings - Fork 100
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
It might be better to change protobuf::libprotobuf to ${Protobuf_LIBRARIES}, becasue protobuf::libprotobuf doesn't work in the lower version of cmake #59
Comments
Thanks! I'll fix it soon |
Fixed in #60, merge after CI passes |
BTW, have you joined dabnn QQ group, 1021964010? |
#60 is merged, could you please verify it on your side? thanks! |
I have changed it. Thank you very much. dabnn is a very useful tool. |
Thanks! |
Thanks! |
In
tools/onnx2bnn/CMakeLists.txt
, it might be better to change protobuf::libprotobuf to ${Protobuf_LIBRARIES}, becasue protobuf::libprotobuf doesn't work in the lower version of cmake, util cmake 3.9.The text was updated successfully, but these errors were encountered: