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

Run on void linux in CI #102

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

Run on void linux in CI #102

wants to merge 8 commits into from

Conversation

mohsen1
Copy link
Collaborator

@mohsen1 mohsen1 commented Feb 16, 2025

No description provided.

- Use updated repository URL for xbps-install to resolve SSL certificate issues.
- Install Rust using rustup inside the Void Linux Docker container.
- Ensure tests run successfully in the Void Linux environment.
@Eloitor
Copy link

Eloitor commented Feb 16, 2025

@mohsen1 can you reproduce the error when running this? (on any system)

cargo test
cargo test lib

And let me know if you need help setting up the void CI

@mohsen1
Copy link
Collaborator Author

mohsen1 commented Feb 17, 2025

@Eloitor no, I am using a Mac. on Ubuntu and Mac it is passing.

yes! please help me run the test in a void linux container. This branch a good start to do this in CI

@Eloitor
Copy link

Eloitor commented Feb 17, 2025

Strange... I tried in a mac with an M4 chip and it failed too.

I used a fresh repo:

git clone https://github.com/bodo-run/yek
cd yek
cargo install --path .
cargo test
cargo test lib

It's strange... I tried again on the mac and I can't reproduce it.

@Eloitor
Copy link

Eloitor commented Feb 17, 2025

To install rust sudo xbps-install -Sy cargo

@Eloitor
Copy link

Eloitor commented Feb 17, 2025

I think it is non deterministic... some times it fails and sometimes doesn't.

- Install base-devel and other required dependencies

- Install Rust using rustup instead of xbps

- Remove unnecessary sudo command
@bodo-run bodo-run deleted a comment from codeautopilot bot Feb 23, 2025
- Replace source with explicit PATH export

- Add version checks for debugging

- Use /root as home directory
- Add multiple repository mirrors for redundancy

- Ensure consistent platform usage with linux/amd64

- Add additional build dependencies

- Enable full backtrace for better error reporting
@mohsen1
Copy link
Collaborator Author

mohsen1 commented Feb 23, 2025

@Eloitor I spent some time trying to run on Void Linux in CI to see what the problem is but this seems like something that can take much longer. Please make a PR with CI changes that tests on Void Linux and if it's failing due to Linux distribution we can take a look.

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