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

Fix running tests #47

Merged
merged 2 commits into from
Dec 15, 2024
Merged

Fix running tests #47

merged 2 commits into from
Dec 15, 2024

Conversation

teqwve
Copy link
Contributor

@teqwve teqwve commented Dec 1, 2024

Thix PR fixes building boxes and running tests. The tests may still not work depending on what compiler is present on host machine, we'll deal with this (move compiling to python with a fixed gcc version) in a separate PR.

@teqwve teqwve requested a review from Wolf480pl December 1, 2024 01:18
@teqwve teqwve force-pushed the fix-tests branch 2 times, most recently from 411f442 to 0dbb614 Compare December 1, 2024 01:34
Packages used in boxes have fall out of ftp.debian.org.

Co-Authored-By: [email protected]
Switch to unittest as nose has been depracated for a while and it's not
really used here, unittest is more than enough.
@teqwve teqwve force-pushed the fix-tests branch 3 times, most recently from eada774 to cd9cb0f Compare December 1, 2024 11:31
Copy link
Member

@Wolf480pl Wolf480pl left a comment

Choose a reason for hiding this comment

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

Can you put the first 2 commits in a separate PR, so that we can merge them before the rest of the PR?

README.md Outdated Show resolved Hide resolved
test/src/compiler/docker-gcc.sh Outdated Show resolved Hide resolved
test/src/compiler/Dockerfile Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@teqwve
Copy link
Contributor Author

teqwve commented Dec 15, 2024

Can you put the first 2 commits in a separate PR, so that we can merge them before the rest of the PR?

Sure, done. We'll create a seaprate PR making test programs compiler version independent from host's.

@teqwve teqwve marked this pull request as ready for review December 15, 2024 13:32
@teqwve teqwve merged commit deed83e into sio2project:master Dec 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants