Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove unused files #19707

Merged
merged 2 commits into from
May 8, 2024
Merged

Remove unused files #19707

merged 2 commits into from
May 8, 2024

Conversation

MiYanni
Copy link
Member

@MiYanni MiYanni commented May 7, 2024

Summary

CC @rbhanda @leecow

@MiYanni MiYanni requested a review from a team May 7, 2024 18:22
@MiYanni MiYanni requested a review from rbhanda as a code owner May 7, 2024 18:22
Copy link
Member

@nagilson nagilson left a comment

Choose a reason for hiding this comment

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

LGTM, but it'd be good to get a review from another person.

@MiYanni MiYanni requested a review from leecow May 7, 2024 20:50
@ViktorHofer ViktorHofer merged commit 51967d4 into dotnet:main May 8, 2024
15 checks passed
jonathanpeppers added a commit to jonathanpeppers/xamarin-android that referenced this pull request May 15, 2024
…4262.2

Context: dotnet/installer#19707
Changes: dotnet/installer@fa261b9...1741345
Changes: dotnet/runtime@4326eb7...84b3339
Changes: dotnet/cecil@4abe3e6...7a4a59f

dotnet/installer is no longer producing a
`Microsoft.Dotnet.Sdk.Internal` "package" for the SDK. This is causing
the maestro code-flow to fail, such as:

    > darc update-dependencies --id 225276
    Looking up build with BAR id 225276
    Checking for coherency updates...
    Using 'Strict' coherency mode. If this fails, a second attempt utilizing 'Legacy' Coherency mode will be made.
    Found no dependencies to update.

We use `Microsoft.Dotnet.Sdk.Internal` to provision the .NET SDK,
similar to:

    ./dotnet-install.sh --version $(MicrosoftDotnetSdkInternalPackageVersion)

Where `eng/Versions.props` is updated by the Maestro bot for the
version number.

Looking for the `Microsoft.Dotnet.Sdk.Internal` dependency, such as:

    > darc gather-drop --id 225276 -o drop --dry-run | grep Microsoft.Dotnet.Sdk.Internal
    Root build - Build number 20240512.2 of
    https://dev.azure.com/dnceng/internal/_git/dotnet-installer
    @ 1741345c6399ae203d8f259fb12fb873dac5129d

But we can find the package for .NET 9 Preview 4:

    > darc gather-drop --id 225611 -o drop --dry-run | grep Microsoft.Dotnet.Sdk.Internal
    Root build - Build number 20240515.4 of
    https://dev.azure.com/dnceng/internal/_git/dotnet-installer
    @ df80b5eb607242b1d8ded158ec97a25e5d5e5e05
    Skipping non-shipping asset Microsoft.Dotnet.Sdk.Internal@9.0.100-preview.4.24265.4

For now, we can use `VS.Tools.Net.Core.SDK.Resolver` instead, as
this is a component inserted into Visual Studio that contains the same
version number.

We may have to change this again in the future, as dotnet/installer is
in the process of merging and/or moving to dotnet/sdk.

MS employees can see more details in the [MS Teams thread][0].

[0]: https://teams.microsoft.com/l/message/19:afba3d1545dd45d7b79f34c1821f6055@thread.skype/1715789991637?tenantId=72f988bf-86f1-41af-91ab-2d7cd011db47&groupId=4d73664c-9f2f-450d-82a5-c2f02756606d&parentMessageId=1715789991637&teamName=.NET%20Core%20Eng%20Services%20Partners&channelName=First%20Responders&createdTime=1715789991637
jonathanpeppers added a commit to dotnet/android that referenced this pull request May 16, 2024
…4262.2 (#8949)

Context: dotnet/installer#19707
Changes: dotnet/installer@fa261b9...1741345
Changes: dotnet/runtime@4326eb7...84b3339
Changes: dotnet/cecil@4abe3e6...7a4a59f

dotnet/installer is no longer producing a
`Microsoft.Dotnet.Sdk.Internal` "package" for the SDK. This is causing
the maestro code-flow to fail, such as:

    > darc update-dependencies --id 225276
    Looking up build with BAR id 225276
    Checking for coherency updates...
    Using 'Strict' coherency mode. If this fails, a second attempt utilizing 'Legacy' Coherency mode will be made.
    Found no dependencies to update.

We use `Microsoft.Dotnet.Sdk.Internal` to provision the .NET SDK,
similar to:

    ./dotnet-install.sh --version $(MicrosoftDotnetSdkInternalPackageVersion)

Where `eng/Versions.props` is updated by the Maestro bot for the
version number.

There is no `Microsoft.Dotnet.Sdk.Internal` package from the build,
such as:

    > darc gather-drop --id 225276 -o drop --dry-run | grep Microsoft.Dotnet.Sdk.Internal
    Root build - Build number 20240512.2 of
    https://dev.azure.com/dnceng/internal/_git/dotnet-installer @ 1741345c6399ae203d8f259fb12fb873dac5129d

But we can find the package for .NET 9 Preview 4:

    > darc gather-drop --id 225611 -o drop --dry-run | grep Microsoft.Dotnet.Sdk.Internal
    Root build - Build number 20240515.4 of
    https://dev.azure.com/dnceng/internal/_git/dotnet-installer @ df80b5eb607242b1d8ded158ec97a25e5d5e5e05
    Skipping non-shipping asset Microsoft.Dotnet.Sdk.Internal@9.0.100-preview.4.24265.4

For now, we can use `VS.Tools.Net.Core.SDK.Resolver` instead, as
this is a component inserted into Visual Studio that contains the same
version number.

We may have to change this again in the future, as dotnet/installer is
in the process of merging and/or moving to dotnet/sdk.

MS employees can see more details in the [MS Teams thread][0].

[0]: https://teams.microsoft.com/l/message/19:afba3d1545dd45d7b79f34c1821f6055@thread.skype/1715789991637?tenantId=72f988bf-86f1-41af-91ab-2d7cd011db47&groupId=4d73664c-9f2f-450d-82a5-c2f02756606d&parentMessageId=1715789991637&teamName=.NET%20Core%20Eng%20Services%20Partners&channelName=First%20Responders&createdTime=1715789991637
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants