search

edsrzf / mmap-go

A portable mmap package for Go
BSD 3-Clause "New" or "Revised" License
938 stars 129 forks source link

Added Power Support ppc64le #26

Closed nagesh4193 closed 2 years ago

nagesh4193 commented 4 years ago

Added power support for the travis.yml file with ppc64le. This is part of the Ubuntu distribution for ppc64le. This helps us simplify testing later when distributions are re-building and re-releasing. Mac and Windows jobs are not available for power support as it is not supported for ppc64le.

nagesh4193 commented 3 years ago

Team - Can someone please look into this and help with the merging. Thanks.

edsrzf commented 2 years ago

I've switched away from Travis to GitHub Actions, so this PR no longer merges.

I'm not sure I understand the goal of this PR. Is it to gain confidence that this package works on PowerPC? Generally I'd expect it to work for any CPU that Go and the golang.org/x/sys package support. The harder thing is adding more operating systems and testing them.

Closing for now. Happy to continue the conversation, but I'm not sure how we'd continue this on GitHub Actions as it does not seem to support different architectures for GitHub-hosted runners.