Skip to content

[src] Don't generate a reference assembly for both the 32-bit and 64-bit variant of Xamarin.WatchOS.dll.#10136

Merged
rolfbjarne merged 2 commits intodotnet:mainfrom
rolfbjarne:src-only-64bit-ref
Nov 23, 2020
Merged

[src] Don't generate a reference assembly for both the 32-bit and 64-bit variant of Xamarin.WatchOS.dll.#10136
rolfbjarne merged 2 commits intodotnet:mainfrom
rolfbjarne:src-only-64bit-ref

Conversation

@rolfbjarne
Copy link
Member

We only need one version, and in any case the code would write both the same
file, causing corruption and random build problems later.

Fixes this problem while building .NET packages:

/Users/builder/jenkins/workspace/xamarin-macios/xamarin-macios/dotnet/package/common.csproj(75,5): error : Classification matches no files: Xamarin.WatchOS.dll [/Users/builder/jenkins/workspace/xamarin-macios/xamarin-macios/dotnet/package/Microsoft.watchOS.Ref/package.csproj]
make[1]: *** [nupkgs/Microsoft.watchOS.Ref.7.1.100-ci.main.35+sha.a4a1fea86.nupkg] Error 1

…bit variant of Xamarin.WatchOS.dll.

We only need one version, and in any case the code would write both the same
file, causing corruption and random build problems later.

Fixes this problem while building .NET packages:

    /Users/builder/jenkins/workspace/xamarin-macios/xamarin-macios/dotnet/package/common.csproj(75,5): error : Classification matches no files: Xamarin.WatchOS.dll [/Users/builder/jenkins/workspace/xamarin-macios/xamarin-macios/dotnet/package/Microsoft.watchOS.Ref/package.csproj]
    make[1]: *** [nupkgs/Microsoft.watchOS.Ref.7.1.100-ci.main.35+sha.a4a1fea86.nupkg] Error 1
@rolfbjarne rolfbjarne added the not-notes-worthy Ignore for release notes label Nov 23, 2020
@monojenkins
Copy link
Contributor

Build failure
Build failed or was aborted

Provisioning succeeded
🔥 Build failed 🔥

@monojenkins
Copy link
Contributor

Build success
Provisioning succeeded
Build succeeded
API Diff (from stable)
API Diff (from PR only) (no change)
Generator Diff (no change)
Test run succeeded

@rolfbjarne rolfbjarne merged commit e241277 into dotnet:main Nov 23, 2020
dalexsoto pushed a commit to dalexsoto/xamarin-macios that referenced this pull request Nov 23, 2020
…bit variant of Xamarin.WatchOS.dll. (dotnet#10136)

* [src] Don't generate a reference assembly for both the 32-bit and 64-bit variant of Xamarin.WatchOS.dll.

We only need one version, and in any case the code would write both the same
file, causing corruption and random build problems later.

Fixes this problem while building .NET packages:

    /Users/builder/jenkins/workspace/xamarin-macios/xamarin-macios/dotnet/package/common.csproj(75,5): error : Classification matches no files: Xamarin.WatchOS.dll [/Users/builder/jenkins/workspace/xamarin-macios/xamarin-macios/dotnet/package/Microsoft.watchOS.Ref/package.csproj]
    make[1]: *** [nupkgs/Microsoft.watchOS.Ref.7.1.100-ci.main.35+sha.a4a1fea86.nupkg] Error 1

* Fix the make rule too so that it doesn't claim it can generate the reference assembly.
dalexsoto pushed a commit to dalexsoto/xamarin-macios that referenced this pull request Dec 3, 2020
…bit variant of Xamarin.WatchOS.dll. (dotnet#10136)

* [src] Don't generate a reference assembly for both the 32-bit and 64-bit variant of Xamarin.WatchOS.dll.

We only need one version, and in any case the code would write both the same
file, causing corruption and random build problems later.

Fixes this problem while building .NET packages:

    /Users/builder/jenkins/workspace/xamarin-macios/xamarin-macios/dotnet/package/common.csproj(75,5): error : Classification matches no files: Xamarin.WatchOS.dll [/Users/builder/jenkins/workspace/xamarin-macios/xamarin-macios/dotnet/package/Microsoft.watchOS.Ref/package.csproj]
    make[1]: *** [nupkgs/Microsoft.watchOS.Ref.7.1.100-ci.main.35+sha.a4a1fea86.nupkg] Error 1

* Fix the make rule too so that it doesn't claim it can generate the reference assembly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

not-notes-worthy Ignore for release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants