-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[workloads] Add net current to current names to make them match in net10 #103050
Conversation
@@ -254,119 +254,119 @@ | |||
"osx-arm64": "Microsoft.NETCore.App.Runtime.AOT.osx-arm64.Cross.android-arm64" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does the alias need the {NetVersion}
too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
probably need to make sure we add aliases where there weren't before and I need to make sure I understand what is going on with the win runtime packs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we had the netversion in any aliases before, it was just the pack names themselves (as this was for VS authoring)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good, but we should run an official build as well to really check.
@lewing This only adds the net9 labels. Will there be a process to port the PR you have for net8 to add those same labels into main (as that PR won't port without fixing conflicts)? |
We discussed this offline but I'll follow up here too incase others have the same question. The net8 labels are already changed in main, the issue is that before this pr the names for the release branches wouldn't have the embedded change |
@lewing I don't think I've seen a successful official build since this merged. First there were all sorts of things wrong with it, but these days it stabilized to:
I'm not sure if this PR is to blame because there were so many other issues around the time it merged, but it looks likely. |
That error was fixed a while back, what build are you looking at? |
Sorry, I've lost it. We have brand new official build issues now but official build hasn't really worked in a week so I'm just trying to figure out what are all the things that need to be fixed in parallel. Looks like this one shouldn't be an issue anymore then. |
There are a lot of people trying to get the official build passing again, hitting a lot of infrastructure problems at the moment. |
No description provided.