-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
configure: Give a better error when the local rustc version is too old #47872
Comments
your system installed rustc is too old. It doesn't support the |
On 01/30/18 11:38, Oliver Schneider wrote:
> configure: build.rustc := /usr/bin/rustc
your system installed rustc is too old. It doesn't support the `..=` syntax yet. There's always a special nightly that is used, but you can probably take any recent one
Thank you for the hint. rustc 1.21.0 is installed.
```
$ rustc --version
rustc 1.21.0
```
I am really irritated, why two version less should be too old to build
the current rustc release.
I’ll try 1.22.0 first.
|
So the configure script should really be fixed to check for the versions it needs to successfully build. |
Because each version is built by the previous one. That's simply the policy.
Well it does ;) it downloads the correct version by default. You explicitly opted to use your system's rustc with |
On 01/30/18 12:32, Oliver Schneider wrote:
> I am really irritated, why two version less should be too old to build
Because each version is built by the previous one. That's simply the policy.
Understood. I didn’t know this policy.
> So the configure script should really be fixed to check for the versions it needs to successfully build.
Well it does ;) it downloads the correct version by default. You explicitly opted to use your system's rustc with `--enable-local-rust`
The switch description is below.
--enable-local-rust use an installed rustc rather than downloading a snapshot
I understand it the way, that it should look for an installed rustc and
use it, if it’s there and usable. There is no point in allowing some
compiler version, and the build fails. Such a switch should have *force*
in it.
|
makes sense. Line 364 in 679f30e
makes it seem like there should be some checking going on? |
error: expected one of
) or
,, found
=``
Mentoring instructions: change the code around Lines 554 to 557 in 006ca9b
compiler in src/stage0.json . It may be helpful to land 9307bb2 first so you can reuse the CompilerMetadata struct.
|
Huh? That local-rebuild case is important for distros, and will not be the same version as |
@cuviper sorry, I was unclear - if it's not the same as stage0.json and it's not the same as src/version. Those are the only 2 versions that are supported. |
fixed in #112231 |
The text was updated successfully, but these errors were encountered: