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
Allow install scripts to be used by koreader CI #71
Conversation
2b2de72
to
83e6fe2
Compare
Please review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Use latest busted dont install luacov-coveralls & luasec: we don't use them at all There is no need to explicitly install dependencies
Closes: #70 |
Btw, I just realized luasec was probably in there because the sync server came first. Cross-ref to koreader/koreader#8607 |
@@ -1,30 +1,11 @@ | |||
#!/usr/bin/env bash | |||
|
|||
HOME=/home/ko | |||
apt-get install --no-install-recommends luarocks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@yparitcher This cannot have ever worked, can it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, it cannot.
If we want to use distro luarocks, it should be done in https://github.com/koreader/virdevenv/blob/11b62a5722b3adc1d98ce8484b89968b136db4c5/docker/ubuntu/baseimage/bootstrap.sh really
But you could do it here with sudo in theory, see
USER ko |
Keep in mind we purposefully remove the apt cache lines earlier
rm -rf /var/lib/apt/lists/* /tmp/* /var/tmp/* |
This is part of work to also use Github actions as CI
Use distro luarocks
Install packages globally
Use latest busted
Dont install luacov-coveralls & luasec: we don't use them at all
There is no need to explicitly install luarocks dependencies
This change is