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

Backport #50832 to v1.10 #56355

Closed
wants to merge 1 commit into from
Closed

Conversation

mofeing
Copy link
Contributor

@mofeing mofeing commented Oct 27, 2024

fixes #55933

This backports #50832 to v1.10.

@mofeing mofeing changed the title Subtype: bug fix for bounds with deeper covariant var (#50832) Backport ebae716 to v1.10 Oct 27, 2024
@mofeing mofeing changed the title Backport ebae716 to v1.10 Backport #50832 to v1.10 Oct 27, 2024
@mofeing
Copy link
Contributor Author

mofeing commented Oct 27, 2024

CC @wsmoses

@mofeing
Copy link
Contributor Author

mofeing commented Oct 27, 2024

Failed tests seem unrelated to this PR:

Error in testset Pkg:
Test Failed at /cache/build/tester-amdci4-10/julialang/julia-release-1-dot-10/julia-bd32e389dd/share/julia/stdlib/v1.10/Pkg/test/new.jl:803
  Expression: ((Pkg.dependencies())[exuuid]).version == v"0.5.3"
   Evaluated: v"0.5.5" == v"0.5.3"
Error in testset Pkg:
Test Failed at /cache/build/tester-amdci4-10/julialang/julia-release-1-dot-10/julia-bd32e389dd/share/julia/stdlib/v1.10/Pkg/test/pkg.jl:279
  Expression: ((Pkg.dependencies())[TEST_PKG.uuid]).version > old_v
   Evaluated: v"0.5.5" > v"0.5.5"
Error in testset Pkg:
Test Failed at /cache/build/tester-amdci4-10/julialang/julia-release-1-dot-10/julia-bd32e389dd/share/julia/stdlib/v1.10/Pkg/test/repl.jl:86
  Expression: ((Pkg.dependencies())[TEST_PKG.uuid]).version > v
   Evaluated: v"0.5.5" > v"0.5.5"

@giordano
Copy link
Contributor

That's because you're branch is old and doesn't have the backport of Pkg.jl which fixed that.

@DilumAluthge
Copy link
Member

@mofeing I think that this should target the backports-release-1.10 branch.

@DilumAluthge DilumAluthge changed the base branch from release-1.10 to kc/backports-release-1.10 October 27, 2024 22:51
@DilumAluthge
Copy link
Member

Agh I changed the target branch, but Git did not like that.

@wsmoses
Copy link
Contributor

wsmoses commented Oct 27, 2024

Can you just reset the branch to back ports and cherry pick the commit

@giordano
Copy link
Contributor

@mofeing I think that this should target the backports-release-1.10 branch.

That doesn't exist anymore.

Agh I changed the target branch, but Git did not like that.

Probably because it's the wrong branch.

@DilumAluthge DilumAluthge changed the base branch from kc/backports-release-1.10 to release-1.10 October 27, 2024 22:55
@giordano
Copy link
Contributor

Isn't it easier to label #50832 as backport to 1.10 and wait for the next v1.10 release to handle all of this automatically?

@mofeing
Copy link
Contributor Author

mofeing commented Oct 27, 2024

That's because you're branch is old and doesn't have the backport of Pkg.jl which fixed that.

@giordano mmm I just updated my release-1.10 branch and cherry-picked the commit again, but same error. GitHub just trolled me.

Isn't it easier to label #50832 as backport to 1.10 and wait for the next v1.10 release to handle all of this automatically?

Does it mean that I shouldn't do anything else and just close this PR?

@giordano
Copy link
Contributor

Does it mean that I shouldn't do anything else and just close this PR?

Yes

@mofeing mofeing closed this Oct 27, 2024
@mofeing mofeing deleted the release-1.10 branch October 27, 2024 23:26
@giordano
Copy link
Contributor

For the benefit of future readers, #50832 was automatically added to the list of backports for v1.10.7 in #56381

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.

5 participants