search

raszi / node-tmp

Temporary file and directory creator for node.js
MIT License
732 stars 93 forks source link

Adds travis jobs on ppc64le #261

Closed dineshks1 closed 3 years ago

dineshks1 commented 3 years ago

This PR adds power support on travis.

silkentrance commented 3 years ago

@dineshks1 Tmp does not have any native artifacts except for its node requirement.

Just out of curiosity, why would you need such a build job?

silkentrance commented 3 years ago

@dineshks1 I still require you to provide an answer as to why we would need the extra build target. Otherwise this PR will be closed without merging.

dineshks1 commented 3 years ago

This is a part of Ubuntu distribution for ppc64le architecture. It will help us to simplify testing later when distributions are re-building and re-releasing.

dineshks1 commented 3 years ago

Please review this merge. For more info tag @gerrith3.

silkentrance commented 3 years ago

@dineshks1 It was never merged. There are no CPU specific dependencies in tmp whatsoever. Why do you not just use the standard ci builds?

Or to be precise, why would I care about downstream build systems?

And to be frank, I do not even care about the Debian SID ripoff called UBUNTU.

silkentrance commented 3 years ago

current and future travis-ci.org is taking hours to have a ci build, so, no, we will not include this.