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

Refactoring CI to use librecores-ci-openrisc docker image #14

Open
wants to merge 3 commits into
base: master
from

Conversation

@Nancy-Chauhan
Copy link

commented Jul 20, 2019

This PR resolves the issue of installing the or1k-tests again and again for every Job by using the librecores-ci-openrisc docker image https://github.com/librecores/docker-images/tree/master/librecores-ci-openrisc
CC @stffrdhrn @oleg-nenashev

@Nancy-Chauhan

This comment has been minimized.

Copy link
Author

commented Jul 23, 2019

@stffrdhrn @oleg-nenashev I have added documentation !

Nancy Chauhan Nancy Chauhan

@Nancy-Chauhan Nancy-Chauhan force-pushed the Nancy-Chauhan:Refactoring branch from 03ebd91 to 6166041 Jul 23, 2019

Show resolved Hide resolved .travis/test.sh
@@ -1,13 +1,12 @@
#!/bin/bash

apt-get update && apt-get install -y curl

export TRAVIS_BUILD_DIR=/src
export HOME=/tmp

This comment has been minimized.

Copy link
@stffrdhrn

stffrdhrn Jul 25, 2019

Member

Why hardcode home to tmp? If it's needed maybe it should be in the docker image?

This comment has been minimized.

Copy link
@Nancy-Chauhan

Nancy-Chauhan Jul 29, 2019

Author

@stffrdhrn https://github.com/librecores/docker-images/blob/master/librecores-ci-openrisc/Dockerfile#L6 , I have added it already in dockerfile , but without export HOME=/tmp in test.sh , test seems to fail , as if it doesn't pick that from dockerfile .

@Nancy-Chauhan Nancy-Chauhan force-pushed the Nancy-Chauhan:Refactoring branch from d357837 to 94b51aa Jul 29, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.