Skip to content

Use Scons in CI with container #959

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

Merged
merged 38 commits into from
Dec 2, 2021
Merged
Changes from all commits
Commits
Show all changes
38 commits
Select commit Hold shift + click to select a range
3eccf33
Update build.yml
ntindle Nov 30, 2021
a826b08
attempt to log data about why we can't see cargo
ntindle Nov 30, 2021
314d296
try and hope
ntindle Nov 30, 2021
d0dac12
Testing without sudo
Amaras Nov 30, 2021
99f119e
Trying to run SCons (Without Cargo?)
Amaras Nov 30, 2021
d67c5f1
fix: actually source cargo
ntindle Dec 1, 2021
0a8febc
ci: attempt to specify shell
ntindle Dec 1, 2021
e45fb53
ci: take another
ntindle Dec 1, 2021
fe5588a
ci: keep testing
ntindle Dec 1, 2021
7238860
ci: test some more
ntindle Dec 1, 2021
10e6573
Trying to "source" cargo in the profile
Amaras Dec 1, 2021
113723f
Correctly trying to source this time
Amaras Dec 1, 2021
ab40763
exporting PATH rather than sourcing in the profile
Amaras Dec 2, 2021
4eb3148
Testing /root/.cargo/bin
Amaras Dec 2, 2021
5633e3b
"Better" diagnostics
Amaras Dec 2, 2021
2fd8630
Fusing path manipulation and running
Amaras Dec 2, 2021
a125ace
feat: use the correct folder for cargo
ntindle Dec 2, 2021
20302ec
ci: log some things
ntindle Dec 2, 2021
c76b1d7
fix: remove failing line, add new bash param
ntindle Dec 2, 2021
828af01
some changes, I guess
ntindle Dec 2, 2021
abdb90f
six question marks
ntindle Dec 2, 2021
433f784
test printing environment after the . thing
ntindle Dec 2, 2021
1c53738
lie about being home
ntindle Dec 2, 2021
ed70197
its hard to build a house
ntindle Dec 2, 2021
a690bd7
consider moving house
ntindle Dec 2, 2021
e70e821
narrow it down?
ntindle Dec 2, 2021
353aa23
secure the property
ntindle Dec 2, 2021
e4c8524
oops homes have rules
ntindle Dec 2, 2021
8b69dce
test the quote
ntindle Dec 2, 2021
d0336ad
fix: redefine HOME from github runner
ntindle Dec 2, 2021
8741d9a
Merge branch 'main' into use-container-scons
ntindle Dec 2, 2021
079ce6d
just set home everywhere?
ntindle Dec 2, 2021
98924b0
Remove everything and try and make it pretty
ntindle Dec 2, 2021
592008c
move the home to another place
ntindle Dec 2, 2021
1ebb8d1
Try and separate Scons and cargo init
ntindle Dec 2, 2021
fa92a01
Go back to the painful solution
ntindle Dec 2, 2021
341ccf6
Merge branch 'main' into use-container-scons
ntindle Dec 2, 2021
cd40be6
Merge branch 'main' into use-container-scons
leios Dec 2, 2021
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 13 additions & 0 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,12 @@ on: pull_request
jobs:
build:
runs-on: ubuntu-latest
container:
options: --entrypoint /bin/bash --user 0
image: ghcr.io/algorithm-archivists/aaa-langs:latest
defaults:
run:
shell: bash --rcfile /root/.bashrc -eo pipefail {0}
steps:
- name: Checkout
uses: actions/checkout@v2
Expand All @@ -14,3 +20,10 @@ jobs:
run: |
npm install
npx honkit build

- name: Initalize cargo and run SCons
env:
HOME: /root
run: |
. "$HOME/.cargo/env"
scons -Q