Skip to content

Adding ppc64le architecture support on travis-ci#76

Merged
Seldaek merged 1 commit intoSeldaek:masterfrom
kishorkunal-raj:master
Nov 6, 2020
Merged

Adding ppc64le architecture support on travis-ci#76
Seldaek merged 1 commit intoSeldaek:masterfrom
kishorkunal-raj:master

Conversation

@kishorkunal-raj
Copy link
Copy Markdown
Contributor

Hi,
I had added ppc64le(Linux on Power) architecture support on Travis-CI in the PR and looks like its been successfully added. I believe it is ready for the final review and merge. The Travis-CI build logs can be verified from the link below.
https://travis-ci.com/github/kishorkunal-raj/jsonlint/builds/198717338

Reason behind running tests on ppc64le: This package is included in the ppc64le versions of RHEL and Ubuntu - this allows the top of tree to be tested continuously as it is for Intel, making it easier to catch any possible regressions on ppc64le before the distros begin their clones and builds. This reduces the work in integrating this package into future versions of RHEL/Ubuntu.

Please have a look.

Regards,
Kishor Kunal Raj

@Seldaek
Copy link
Copy Markdown
Owner

Seldaek commented Nov 6, 2020

This is a pretty simple package, with very little chance of endianness breaking stuff IMO. I'm not against adding it but the main problem I see is that we'd like to migrate to GH Actions instead of travis sooner or later, and I don't think that supports ppc64le.

Anyway merging as I guess it's harmless enough for now.

@Seldaek Seldaek merged commit 7a5f805 into Seldaek:master Nov 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants