ars3niy / tdlib-purple

libpurple Telegram plugin using tdlib
Other
147 stars 29 forks source link

Add support to ppc64le with gcc. Install gettext #100

Open gururajrkatti opened 3 years ago

gururajrkatti commented 3 years ago

Add support to Linux on Power Little Endian architecture. Continuously building/testing on this platform along with intel will help in detecting/fixing the problems in early stage.

ars3niy commented 3 years ago

We seem to replacing travis with github actions though :-|

BenWiederhake commented 3 years ago

Good news everyone, GitHub Actions doesn't support running natively on other architectures. However, it does support running stuff in Docker: https://github.com/marketplace/actions/run-on-architecture#advanced-example

So one way to do this on GA is by creating a new workflow which basically just runs all build steps inside docker. Since we don't use any advanced stuff, this should be a copy-and paste job, since the cache action can run outside of docker. Also, this means we can in theory extend to aarch64, armv7, and s390x, just by extending the matrix.

gururajrkatti commented 3 years ago

@ars3niy thanks for the feedback. Travis does want to continue open source support Pl. refer to https://blog.travis-ci.com/oss-announcement if that is the concern. Meanwhile I can check if it is possible to run these steps as part of a docker,

BenWiederhake commented 3 years ago

The announcement conveniently omits all the crucial parts:

ars3niy commented 3 years ago

I'm a simple person. Ben say github actions, me merge. Now can't merge this one :(

BenWiederhake commented 3 years ago

Since gururajrkatti already wrote this commit twice, I'll have a look at how to write it for Github Actions – I will do that next year, though. Guten Rutsch, as we say in Germany! :)