-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Error when building onnxruntime on Fedora 32 for aarch64 #7520
Comments
Which source code version? Could you please give me a commit id? I just updated the protobuf submodule. Does the same problem also exist in x86_64 build? |
That was on master earlier today, I finished rerunning after running. I didn't keep track of the commit id, but I'll look into it. However I reran as follows:
This time it got to 61% before throwing an error:
I have not yet tested in x86_64 build. |
Your gcc version is gcc 10.2.1 but we only tested our code with GCC 8.x. I guess that's the reason. I will add some tests in CI build to catch such errors. Meantime, you can open build.py and remove this line: |
See protocolbuffers/protobuf#7514 The fix has been checked in. Later on I will bring it in our code base. |
I ran into another problem compiling:
|
I tried the latest master branch code with gcc 10.2, it's all good except the google protobuf issue you mentioned above. |
Describe the bug
Error when building onnxruntime on Fedora 32 for aarch64
onnxruntime/cmake/external/protobuf/src/google/protobuf/stubs/logging.h:188:50: error: comparison of unsigned expression in ‘>= 0’ is always true [-Werror=type-limits]
My build process is here:
Full output is available here:
https://pastebin.com/UpSKi4M7
Urgency
Ideally resolved in a couple of weeks
System information
To Reproduce
Can reproduce in a docker image with QEMU emulator, use
FROM arm64v8/fedora:32
, add dependencies and call:Expected behavior
It should compile
The text was updated successfully, but these errors were encountered: