jonathan-albrecht-ibm commented on pull request #9195: URL: https://github.com/apache/arrow/pull/9195#issuecomment-761176372
> Should we also update our CI scripts for `noasm` case? > > * https://github.com/apache/arrow/blob/master/ci/scripts/go_build.sh > * https://github.com/apache/arrow/blob/master/ci/scripts/go_test.sh Not sure if I can answer this question and apologies if its meant for someone else but I don't think the CI scripts need to change, at least not as part of this PR. `noasm` is not the common case on 390x but since its supported on other archs I wanted to make sure it is on s390x as well. It came up while trying to build another project that depends on arrow that had specified `noasm` and so it didn't build out of the box. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
