esafak commented on code in PR #2357:
URL: https://github.com/apache/fory/pull/2357#discussion_r2159995740
##########
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:
I tried by centralizing flags in
[BUILD](https://github.com/apache/fory/pull/2357/commits/6fba5a08fda246b4c452481635978db4f7b035da#diff-3053677e4333a96b20ed76c662bca07fb2e1ce059479170139209356787da09c)
but failed because
[pyx_library](https://github.com/grpc/grpc/blob/v1.73.0/bazel/cython_library.bzl)
does not have a way to pass `copts` to the cc_binary. If you can suggest
another way we can retry.
--
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]
