Skip to content

Add GNU gcc and binutils #10

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

Merged
merged 1 commit into from
May 29, 2025
Merged

Conversation

psumbera
Copy link
Contributor

@marcoieni
Copy link
Member

We can merge this when the MCP requested in rust-lang/rust#138699 (comment) is approved 👍
Otherwise these files are going to be unused.

@psumbera
Copy link
Contributor Author

@marcoieni
Copy link
Member

marcoieni commented Apr 28, 2025

oh, I see 👍
Then PR welcome to switch to the ci-mirror URLs 👍

@psumbera
Copy link
Contributor Author

We can merge this when the MCP requested in rust-lang/rust#138699 (comment) is approved 👍 Otherwise these files are going to be unused.

MCP was approved (rust-lang/compiler-team#870). Can you please merge this now?

@marcoieni
Copy link
Member

tests aren't passing. can you fix them and force push?

@psumbera
Copy link
Contributor Author

tests aren't passing. can you fix them and force push?

Do you have any idea what can be wrong with it?

error: HTTP status client error (403 Forbidden) for url (https://ftp.gnu.org/gnu/gcc/gcc-8.4.0/gcc-8.4.0.tar.xz)
error: HTTP status client error (403 Forbidden) for url (https://ftp.gnu.org/gnu/binutils/binutils-2.44.tar.xz)

@jieyouxu
Copy link
Member

jieyouxu commented May 27, 2025

Sometimes (more like quite frequently) the GCC servers are unreliable, IIRC they are mirrored for cg_gcc for this exact reason (tho I might be misremembering)

@psumbera
Copy link
Contributor Author

tests aren't passing. can you fix them and force push?

Done. Can you please pull it now?

[[files]]
name = "rustc/gcc-8.4.0.tar.xz"
source = "https://bigsearcher.com/mirrors/gcc/releases/gcc-8.4.0/gcc-8.4.0.tar.xz"
sha256 = "e3a06e52d1e01f27ed55104ad233c3b0d1e99cfb5ff98ab022dc941edd1b2dd4"
Copy link
Member

Choose a reason for hiding this comment

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

I downloaded gcc-8.4.0.tar.xz and the sha doesn't correspond. What am I doing wrong? 🤔

$ sha256 gcc-8.4.0.tar.gz
SHA256 (gcc-8.4.0.tar.gz) = 41e8b145832fc0b2b34c798ed25fb54a881b0cee4cd581b77c7dc92722c116a8

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure. But it must be something wrong on your side. Is it really tar archive?

The only problem I'm confused is that sha256sum tells me:
e30a6e52d10e1f27ed55104ad233c30bd1e99cfb5ff98ab022dc941edd1b2dd4
While your CI thinks it's:
e3a06e52d1e01f27ed55104ad233c3b0d1e99cfb5ff98ab022dc941edd1b2dd4
^^ (swaped characters?!)

Copy link
Member

Choose a reason for hiding this comment

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

I downloaded the tar from https://gcc.gnu.org/pub/gcc/releases/gcc-8.4.0/gcc-8.4.0.tar.xz not the mirror.
Can you calculate the sha256 on your laptop of the release on the gnu website?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I downloaded the tar from https://gcc.gnu.org/pub/gcc/releases/gcc-8.4.0/gcc-8.4.0.tar.xz not the mirror. Can you calculate the sha256 on your laptop of the release on the gnu website?

I already did it. It's the same as from mirror site.

Copy link
Member

Choose a reason for hiding this comment

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

oh, I downloaded .gz instead of .xz
I also have swapped characters locally. not sure why

@marcoieni marcoieni added this pull request to the merge queue May 29, 2025
Merged via the queue into rust-lang:main with commit 63484a5 May 29, 2025
3 checks passed
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.

3 participants