search

praiskup / argparse-manpage

Automatically build man-pages for your Python project
Apache License 2.0
40 stars 22 forks source link

Added Power Support ppc64le #30

Closed nagesh4193 closed 3 years ago

nagesh4193 commented 3 years ago

Added power support for the travis.yml file with ppc64le

praiskup commented 3 years ago

The test matrix is already pretty large (6 tasks). Does it make sense to add everything once more for ppc64le? Can we e.g. add one test only, against the newest stable python? I'm just afraid to consume too much Travis resources...

Note that argparse-manpage package shouldn't suffer from architecture problems, it's so called "noarch"... you can build it on x86_64, and then run on ppc64le. What is to motivation for this PR?

praiskup commented 3 years ago

I forgot to thank you for the PR!

nagesh4193 commented 3 years ago

@praiskup - This is part of the Ubuntu distribution for ppc64le. This helps us simplify testing later when distributions are re-building and re-releasing. Mostly the changes of power support runs in parallel to Intel binaries without consuming much of the travis resources. Thank you. For more info tag @gerrith3

gerrith3 commented 3 years ago

@praiskup In terms of running the full test matrix on ppc64le - 1) all builds run in parallel, so there shouldn't be any net add to your build/test time 2) the travis on power hardware is free (at least to you ;-) ) and 3) I believe the build/test cycle here is pretty short. But it is your call as to whether we take a subset from your matrix or test the whole matrix. @nagesh4193 can update the .travis.yml whichever way you prefer.

praiskup commented 3 years ago

I wouldn't mind if it took a bit long er to run the tasks, really ... It just sounds a bit wasteful to duplicae everything O:-) Nevermind, let's give it a try.