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

Update binutils to 2.38 on illumos and solaris #1641

Merged
merged 3 commits into from
Mar 8, 2025

Conversation

ahqsoftwares
Copy link
Contributor

@ahqsoftwares ahqsoftwares commented Mar 8, 2025

resolves #1639

@ahqsoftwares ahqsoftwares requested a review from a team as a code owner March 8, 2025 13:25
@ahqsoftwares ahqsoftwares changed the title Update binutils to 2.38, probably #1693 (fix: illumos) Update binutils to 2.38 Mar 8, 2025
@Emilgardis
Copy link
Member

Can you squash the commits? Thanks!

@ahqsoftwares
Copy link
Contributor Author

Actually, I'm not sure how to squash commits. Do I need to create another pr?

@Emilgardis
Copy link
Member

To squash them, you can do git rebase -i HEAD~6 and then set every commit except the first to f or fixup

@ahqsoftwares
Copy link
Contributor Author

I am not sure, is this correct?

@Emilgardis
Copy link
Member

No that made it worse :D I can fix it for you

@Emilgardis
Copy link
Member

/ci try --target *illumos

This comment has been minimized.

@Emilgardis Emilgardis added the CI-x86_64-unknown-illumos Run CI for x86_64-unknown-illumos target label Mar 8, 2025
@briansmith
Copy link
Contributor

Can you do docker/solaris.sh too?

This comment has been minimized.

@Emilgardis
Copy link
Member

/ci try --target *solaris

This comment has been minimized.

@Emilgardis Emilgardis changed the title (fix: illumos) Update binutils to 2.38 Update binutils to 2.38 on illumos and solaris Mar 8, 2025

This comment has been minimized.

@ahqsoftwares
Copy link
Contributor Author

Is the error related to the script or docker? The checksums match on wsl

@Emilgardis
Copy link
Member

trying to parse the make build but can't see any obvious actual error, it's warning about makeinfo so trying now locally after adding texinfo

@Emilgardis
Copy link
Member

/ci try --target *solaris --target *illumos

This comment has been minimized.

Copy link

github-actions bot commented Mar 8, 2025

Try run for comment

Failed Jobs

Successful Jobs

@Emilgardis Emilgardis removed the CI-sparcv9-sun-solaris Run CI for sparcv9-sun-solaris target label Mar 8, 2025
@Emilgardis Emilgardis removed the CI-x86_64-pc-solaris Run CI for x86_64-pc-solaris target label Mar 8, 2025
@Emilgardis
Copy link
Member

__sysconf_xpg7 is part of solaris 11, can't solve that problem in this pr... see #1599

@Emilgardis Emilgardis added this pull request to the merge queue Mar 8, 2025
Merged via the queue into cross-rs:main with commit 36c0d78 Mar 8, 2025
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-x86_64-unknown-illumos Run CI for x86_64-unknown-illumos target
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot compile ring 0.17.13 (also reqwest) due to outdated bintools
3 participants