Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding ppc64le architecture support on travis-ci #5271

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kishorkunal-raj
Copy link

Hi,
I have added support for ppc64le architecture support on travis-ci in the branch. I believe it is ready for the final review and merge.
The travis-ci build log can be tracked on the link : https://travis-ci.com/github/kishorkunal-raj/guava/builds/189450395
Please have a look on it

Thanks !!

@google-cla google-cla bot added the cla: yes label Oct 12, 2020
@cpovirk
Copy link
Member

cpovirk commented Oct 12, 2020

Thanks. I haven't looked in detail at the logs, but I see a timeout in the Travis run here. Maybe we need travis_wait for the install command, similar to what we have here...?

script: travis_wait 60 mvn -B -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn verify -U -Dmaven.javadoc.skip=true -f $ROOT_POM

@cpovirk cpovirk added P3 no SLO package=general type=other Miscellaneous activities not covered by other type= labels labels Oct 12, 2020
@cpovirk
Copy link
Member

cpovirk commented Oct 12, 2020

(We've wanted a convenient way to test on PPC a few times in the past, so this could be handy.)

@cpovirk
Copy link
Member

cpovirk commented Jun 26, 2023

Sorry, I'm not sure that I realized that this PR had gotten updated after the earlier discussion.

Nowadays, it would have to be updated for GitHub CI. I don't know if anyone is interested in giving that a shot, but we should be able to accept a change if someone is.

Copy link
Member

@cpovirk cpovirk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to approve this as a test, even though we'd need to rework it to use GitHub CI if we wanted to merge it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes P3 no SLO package=general type=other Miscellaneous activities not covered by other type= labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants