search

smhg / gettext-parser

Parse and compile gettext po and mo files, nothing more, nothing less
MIT License
160 stars 44 forks source link

Changes in .travis.yml file #57

Closed nageshlop closed 3 years ago

nageshlop commented 3 years ago

This PR adds CI support for the IBM Power Little Endian (ppc64le) architecture. The idea is to ensure that the builds on this architecture are continuously tested along with the Intel builds (amd64) as this is part of the ubuntu distro on that architecture as well and detecting (and fixing) any issues or failures early would help to ensure that we are always up to date. The build and test results are available at the below location. https://travis-ci.com/github/nageshlop/gettext-parser

smhg commented 3 years ago

LGTM, thanks! I've only vaguely heard about Power PC in the past. Isn't it supposed to be outdated? Or am I mixing things up?

seth-priya commented 3 years ago

@smhg thanks for the feedback - little endian support for the IBM Power Platform (ppc64le) was introduced in 2014 and has been gaining momentum since. The latest versions of all the major distros including ubuntu 20.x, RHEL 7/8, Centos 7/8, Fedora, Debian, Alpine have support for ppc64le and we have close to probably 1K communities running this on travis and other CIs. Our team at IBM has been working with several open source communities over the last 6+ years to enhance and increase the overall ecosystem of open source packages on Power. Besides, there are several enterprises who run their workloads on these servers today, hope that helps. Do let me know if I can help with additional context, thank you!

smhg commented 3 years ago

@seth-priya Thank you for the explanation! Please let me clearly say that I fully support adding support to as many architectures as possible. However, it sounds like waste of resources to me to test every individual project on every platform. Let's make sure the intermediate levels Node/NPM/... take care of this. Also, this project will probably move to GitHub Actions soon (which doesn't seem to support this yet). In that light, I'll close this. Feel free to disagree though. I'm happy to reconsider.