Skip to content

[release/9.0.1xx] fix Microsoft.NET.Sdk.Android.Manifest-9.0.100 version band #10112

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

Merged
merged 3 commits into from
May 7, 2025

Conversation

jonathanpeppers
Copy link
Member

In fc72bb4, we migrated to a 9.0.300 .NET SDK, which unfortunately resulted in an Android workload manifest with a 9.0.300 version band:

Microsoft.NET.Sdk.Android.Manifest-9.0.300.35.0.73.nupkg

To stay on 9.0.100, we need to also set $(DotNetSdkManifestsFolder) in eng/Versions.props in addition to the other properties we set in fc72bb4.

…rsion band

In fc72bb4, we migrated to a 9.0.300 .NET SDK, which unfortunately
resulted in an Android workload manifest with a 9.0.300 version band:

    Microsoft.NET.Sdk.Android.Manifest-9.0.300.35.0.73.nupkg

To stay on 9.0.100, we need to also set `$(DotNetSdkManifestsFolder)`
in `eng/Versions.props` in addition to the other properties we set in
fc72bb4.
@jonathanpeppers
Copy link
Member Author

Package name looks correct now:

image

@jonathanpeppers
Copy link
Member Author

I think something is breaking on the Windows lane:

1>InstallManifestAndDependencies:
         Creating directory "C:\a\_work\1\s\bin\Release\lib\sdk-manifests\9.0.100\android.deps.workload\0.0.1\".
         "C:\a\_work\1\s\bin\Release\dotnet\dotnet" workload install android-deps --configfile "C:\a\_work\1\s\NuGet.config" --skip-manifest-update --skip-sign-check --verbosity diag
         Workload installation failed: Workload ID android-deps is not recognized.

I'm going to try a different property.

@jonathanpeppers jonathanpeppers enabled auto-merge (squash) May 7, 2025 14:42
@jonathanpeppers jonathanpeppers merged commit 6068071 into release/9.0.1xx May 7, 2025
57 checks passed
@jonathanpeppers jonathanpeppers deleted the dev/peppers/9.0.100 branch May 7, 2025 15:28
jonathanpeppers added a commit that referenced this pull request May 7, 2025
Backport of: #10112

In fc72bb4, we migrated to a 9.0.300 .NET SDK, which unfortunately
resulted in an Android workload manifest with a 9.0.300 version band:

    Microsoft.NET.Sdk.Android.Manifest-9.0.300.35.0.73.nupkg

To stay on 9.0.100, we need should use `$(DotNetAndroidManifestVersionBand)`
for the manifest version band instead of `$(DotNetSdkManifestsFolder)`.

I also had to make a similar change in `DotNet.targets`.
jonathanpeppers added a commit that referenced this pull request May 7, 2025
Backport of: #10112

In fc72bb4, we migrated to a 9.0.300 .NET SDK, which unfortunately
resulted in an Android workload manifest with a 9.0.300 version band:

    Microsoft.NET.Sdk.Android.Manifest-9.0.300.35.0.73.nupkg

To stay on 9.0.100, we need should use `$(DotNetAndroidManifestVersionBand)`
for the manifest version band instead of `$(DotNetSdkManifestsFolder)`.

I also had to make a similar change in `DotNet.targets`.
@github-actions github-actions bot locked and limited conversation to collaborators Jun 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants