chaokunyang commented on code in PR #2357:
URL: https://github.com/apache/fory/pull/2357#discussion_r2158109837
##########
ci/deploy.sh:
##########
@@ -122,7 +122,15 @@ build_pyfory() {
echo "Install pyfory"
# Fix strange installed deps not found
pip install setuptools -U
- bazel build //:cp_fory_so
+
+ # Detect host architecture and only pass x86_64 config when appropriate
+ ARCH=$(uname -m)
+ if [[ "$ARCH" == "x86_64" || "$ARCH" == "amd64" ]]; then
+ bazel build --config=x86_64 //:cp_fory_so
Review Comment:
It would be great if we can keep all build as `bazel build //:cp_fory_so`,
this will make new contributor more easy to build project
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
