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

[tests] set $(DisableTransitiveFrameworkReferenceDownloads) #8374

Conversation

jonathanpeppers
Copy link
Member

Context: #8360

MSBuild tests that use net7.0-android TargetFrameworks can fail on release branches with:

(Restore target) ->
error NU1102: Unable to find package Microsoft.AspNetCore.App.Ref with version (= 7.0.12)
error NU1102:   - Found 1621 version(s) in dotnet7 [ Nearest version: 7.0.0-rtm.22513.3 ]
error NU1102:   - Found 1537 version(s) in dotnet8 [ Nearest version: 8.0.0-alpha.1.22416.11 ]
error NU1102:   - Found 87 version(s) in testsource1 [ Nearest version: 8.0.0-preview.1.23112.2 ]
error NU1102:   - Found 87 version(s) in dotnet-public [ Nearest version: 8.0.0-preview.1.23112.2 ]
error NU1102:   - Found 0 version(s) in darc-pub-dotnet-emsdk-9506882
error NU1102:   - Found 0 version(s) in darc-pub-dotnet-runtime-26e0f82
error NU1102:   - Found 0 version(s) in darc-pub-dotnet-runtime-301ba1e
error NU1102:   - Found 0 version(s) in dotnet-eng
error NU1102:   - Found 0 version(s) in dotnet8-transport
error NU1102:   - Found 0 version(s) in xamarin.android util
    0 Warning(s)
    1 Error(s)

To solve this, we can set $(DisableTransitiveFrameworkReferenceDownloads) as we did in 2320e20 so it doesn't try to restore ASP.NET or Windows Desktop packs.

The MSBuild integration tests have their own Directory.Build.props, so this seems like a reasonable place to set this value.

Context: xamarin#8360

MSBuild tests that use `net7.0-android` `TargetFrameworks` can fail
on release branches with:

    (Restore target) ->
    error NU1102: Unable to find package Microsoft.AspNetCore.App.Ref with version (= 7.0.12)
    error NU1102:   - Found 1621 version(s) in dotnet7 [ Nearest version: 7.0.0-rtm.22513.3 ]
    error NU1102:   - Found 1537 version(s) in dotnet8 [ Nearest version: 8.0.0-alpha.1.22416.11 ]
    error NU1102:   - Found 87 version(s) in testsource1 [ Nearest version: 8.0.0-preview.1.23112.2 ]
    error NU1102:   - Found 87 version(s) in dotnet-public [ Nearest version: 8.0.0-preview.1.23112.2 ]
    error NU1102:   - Found 0 version(s) in darc-pub-dotnet-emsdk-9506882
    error NU1102:   - Found 0 version(s) in darc-pub-dotnet-runtime-26e0f82
    error NU1102:   - Found 0 version(s) in darc-pub-dotnet-runtime-301ba1e
    error NU1102:   - Found 0 version(s) in dotnet-eng
    error NU1102:   - Found 0 version(s) in dotnet8-transport
    error NU1102:   - Found 0 version(s) in xamarin.android util
        0 Warning(s)
        1 Error(s)

To solve this, we can set `$(DisableTransitiveFrameworkReferenceDownloads)`
as we did in 2320e20 so it doesn't try to restore ASP.NET or Windows
Desktop packs.

The MSBuild integration tests have their own `Directory.Build.props`, so
this seems like a reasonable place to set this value.
@jonathanpeppers jonathanpeppers merged commit 95946d2 into xamarin:main Sep 26, 2023
45 of 47 checks passed
@jonathanpeppers jonathanpeppers deleted the DisableTransitiveFrameworkReferenceDownloadsTests branch September 26, 2023 21:44
jonathanpeppers added a commit that referenced this pull request Sep 26, 2023
Context: #8360

MSBuild tests that use `net7.0-android` `TargetFrameworks` can fail
on release branches with:

    (Restore target) ->
    error NU1102: Unable to find package Microsoft.AspNetCore.App.Ref with version (= 7.0.12)
    error NU1102:   - Found 1621 version(s) in dotnet7 [ Nearest version: 7.0.0-rtm.22513.3 ]
    error NU1102:   - Found 1537 version(s) in dotnet8 [ Nearest version: 8.0.0-alpha.1.22416.11 ]
    error NU1102:   - Found 87 version(s) in testsource1 [ Nearest version: 8.0.0-preview.1.23112.2 ]
    error NU1102:   - Found 87 version(s) in dotnet-public [ Nearest version: 8.0.0-preview.1.23112.2 ]
    error NU1102:   - Found 0 version(s) in darc-pub-dotnet-emsdk-9506882
    error NU1102:   - Found 0 version(s) in darc-pub-dotnet-runtime-26e0f82
    error NU1102:   - Found 0 version(s) in darc-pub-dotnet-runtime-301ba1e
    error NU1102:   - Found 0 version(s) in dotnet-eng
    error NU1102:   - Found 0 version(s) in dotnet8-transport
    error NU1102:   - Found 0 version(s) in xamarin.android util
        0 Warning(s)
        1 Error(s)

To solve this, we can set `$(DisableTransitiveFrameworkReferenceDownloads)`
as we did in 2320e20 so it doesn't try to restore ASP.NET or Windows
Desktop packs.

The MSBuild integration tests have their own `Directory.Build.props`, so
this seems like a reasonable place to set this value.
grendello added a commit to grendello/xamarin-android that referenced this pull request Sep 27, 2023
* main:
  [tests] set `$(DisableTransitiveFrameworkReferenceDownloads)` (xamarin#8374)
@github-actions github-actions bot locked and limited conversation to collaborators Jan 22, 2024
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.

None yet

3 participants