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

CI: Build for x86_64-unknown-linux-gnux32. #464

Closed
wants to merge 1 commit into from

Conversation

briansmith
Copy link
Contributor

@briansmith briansmith commented Jun 7, 2024

We cannot run the tests on the Ubuntu runners but we can build for the target.

@briansmith briansmith marked this pull request as ready for review June 7, 2024 16:37
josephlr added a commit that referenced this pull request Jun 7, 2024
This PR merges the "extra" Linux tests into a single job. This reduces
the number of jobs we have to run as part of the CI. We also explictly
install libc and libgcc (to make it clear these are the only extra dev
dependancies).

I also added building/linking of the x32 target. As noted in #464, this
will not run in the current github actions runner due to an incompatible
kernel config.

Signed-off-by: Joe Richey <[email protected]>
josephlr added a commit to josephlr/getrandom that referenced this pull request Jun 7, 2024
This PR merges the "extra" Linux tests into a single job. This reduces
the number of jobs we have to run as part of the CI. We also explictly
install libc and libgcc (to make it clear these are the only extra dev
dependancies).

I also added building/linking of the x32 target. As noted in rust-random#464, this
will not run in the current github actions runner due to an incompatible
kernel config.

Signed-off-by: Joe Richey <[email protected]>
@josephlr
Copy link
Member

josephlr commented Jun 7, 2024

I like this idea, but we should also link on this target, as it's only running which poses a problem. Closing as part of #466 (which also simplifies some of our Linux cross-compilation).

@josephlr josephlr closed this Jun 7, 2024
josephlr added a commit that referenced this pull request Jun 7, 2024
This PR explictly installs libc and libgcc (to make it clear these are
the only extra dev dependancies).

I also added building/linking of the x32 target. As noted in #464, this
will not run in the current github actions runner due to an incompatible
kernel config.

Signed-off-by: Joe Richey <[email protected]>
josephlr added a commit that referenced this pull request Jun 7, 2024
This PR explictly installs libc and libgcc (to make it clear these are
the only extra dev dependancies).

I also added building/linking of the x32 target. As noted in #464, this
will not run in the current github actions runner due to an incompatible
kernel config.

Signed-off-by: Joe Richey <[email protected]>
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