-
Notifications
You must be signed in to change notification settings - Fork 144
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
Possible inconsistencies with override versions #654
Labels
bug
Something isn't working
Comments
Nice find, this does look like a bug in how overrides work. |
There are 2 problems here:
I'll make 2 PRs that will address each issue. |
alfiedotwtf
added a commit
that referenced
this issue
Sep 28, 2024
alfiedotwtf
added a commit
that referenced
this issue
Sep 29, 2024
alfiedotwtf
added a commit
that referenced
this issue
Sep 29, 2024
alfiedotwtf
added a commit
that referenced
this issue
Oct 4, 2024
alfiedotwtf
added a commit
that referenced
this issue
Oct 4, 2024
alfiedotwtf
added a commit
that referenced
this issue
Oct 4, 2024
alfiedotwtf
added a commit
that referenced
this issue
Oct 4, 2024
alfiedotwtf
added a commit
that referenced
this issue
Oct 4, 2024
alfiedotwtf
added a commit
that referenced
this issue
Oct 4, 2024
alfiedotwtf
added a commit
that referenced
this issue
Oct 4, 2024
alfiedotwtf
added a commit
that referenced
this issue
Oct 4, 2024
alfiedotwtf
added a commit
that referenced
this issue
Oct 4, 2024
JoshuaBatty
pushed a commit
that referenced
this issue
Oct 11, 2024
alfiedotwtf
added a commit
that referenced
this issue
Oct 24, 2024
alfiedotwtf
added a commit
that referenced
this issue
Oct 24, 2024
alfiedotwtf
added a commit
that referenced
this issue
Oct 25, 2024
alfiedotwtf
added a commit
that referenced
this issue
Oct 25, 2024
alfiedotwtf
added a commit
that referenced
this issue
Oct 27, 2024
alfiedotwtf
added a commit
that referenced
this issue
Oct 27, 2024
When a proxy CLI'd command downloads a missing toolchain, multiple binary files within the downloaded distribution were being symlinked to the same one target file (thus overwriting each previous target).
alfiedotwtf
added a commit
that referenced
this issue
Oct 27, 2024
…orc (#654) When checking if a component is distributed by forc, we currently only look at forc's direct executables. This needs to be extended to resolve components, plugins, and executables.
alfiedotwtf
added a commit
that referenced
this issue
Oct 28, 2024
…orc (#654) When checking if a component is distributed by forc, we currently only look at forc's direct executables. This needs to be extended to resolve components, plugins, and executables.
alfiedotwtf
added a commit
that referenced
this issue
Oct 28, 2024
When a proxy CLI'd command downloads a missing toolchain, multiple binary files within the downloaded distribution were being symlinked to the same one target file (thus overwriting each previous target).
alfiedotwtf
added a commit
that referenced
this issue
Oct 28, 2024
Fix incorrect binary link target (#654) When a proxy CLI'd command downloads a missing toolchain, multiple binary files within the downloaded distribution were being symlinked to the same one target file (thus overwriting each previous target).
alfiedotwtf
added a commit
that referenced
this issue
Oct 28, 2024
…orc (#654) When checking if a component is distributed by forc, we currently only look at forc's direct executables. This needs to be extended to resolve components, plugins, and executables.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Problem
Unless I'm mistaken, overrides works like the following:
This doesn't seem to be the case:
The issue #652 is related since we do want an error message here, but I think this is a bug seeing as
forc --version
above didn't acknowledge the overridden version.Edit: related PRs:
Steps
Let's start from scratch:
Now we create the override:
But when we now list the components, I'm expecting
fuel-core
to be at0.17.11
as above but instead see:... notice how the first line above show we are still on
latest-aarch64-apple-darwin
rather thanbeta-3
.Possible Solution(s)
It's only after installing the
beta-3
toolchain do we get the proper output:The problem is either:
Issue #652 deals with a side effect of this bug, but I'm thinking it's a larger issue.
Notes
No response
Fuelup version
Installed components
The text was updated successfully, but these errors were encountered: