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

Updating actions #63

Merged
merged 20 commits into from
Oct 11, 2023
Merged

Updating actions #63

merged 20 commits into from
Oct 11, 2023

Conversation

dgkf
Copy link
Owner

@dgkf dgkf commented Oct 11, 2023

  • Adding testing and linting jobs

- build: x86_64-linux
os: ubuntu-latest
rust: stable
target: x86_64-unknown-linux-gnu
Copy link

Choose a reason for hiding this comment

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

For portability reasons, I recommend using the musl target.
For gnu targets, the binaries built here probably won't work on Debian old stable or Ubuntu 20.04.

@eitsupi eitsupi mentioned this pull request Oct 11, 2023
@dgkf dgkf merged commit 571dbb6 into main Oct 11, 2023
11 checks passed
@dgkf dgkf deleted the 34-build-releases branch October 11, 2023 20:59
Comment on lines +33 to +43
- build: aarch64-linux
os: ubuntu-latest
rust: stable
target: aarch64-unknown-linux-gnu
cross: true

- build: riscv64-linux
os: ubuntu-latest
rust: stable
target: riscv64gc-unknown-linux-gnu
cross: true
Copy link

Choose a reason for hiding this comment

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

How about change these to musl too?

Copy link
Owner Author

Choose a reason for hiding this comment

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

To be honest, I don't have a good sense for the advantages of each. What would be the reasons to favor one over the other?

Copy link
Owner Author

Choose a reason for hiding this comment

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

For riscv64, it seems the toolchain to include rust-std isn't available yet for musl:
https://github.com/dgkf/R/actions/runs/6490904840/job/17627388865?pr=64#step:5:65

Copy link

Choose a reason for hiding this comment

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

Binaries built with gnu targets are not compatible with environments other than those in which they were built.
See, for example, the following problem with mdBook:
rust-lang/mdBook#1862
rust-lang/mdBook#1954

Copy link
Owner Author

Choose a reason for hiding this comment

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

I see - okay, then I'll add it where we can. It seems the riscv64 is still going to have to stay on gnu (at least if we want to use the stable toolchain, not sure about nightly/others) but the others can be migrated.

Copy link

Choose a reason for hiding this comment

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

Thanks!

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