Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.
/ corefx Public archive

Moved Interop Locations For System.IO.Compression.Brotli #36614

Merged
merged 2 commits into from
Apr 7, 2019
Merged

Moved Interop Locations For System.IO.Compression.Brotli #36614

merged 2 commits into from
Apr 7, 2019

Conversation

eanova
Copy link

@eanova eanova commented Apr 4, 2019

This commit is to relocate the project's interop definitions into the Common library project.

@eanova
Copy link
Author

eanova commented Apr 5, 2019

According to the stacktraces both checks fail during testing of System.Text.Json.Tests, not quite sure what the next step would be. Is there a way to rerun the tests?

@ahsonkhan
Copy link
Member

According to the stacktraces both checks fail during testing of System.Text.Json.Tests, not quite sure what the next step would be. Is there a way to rerun the tests?

https://github.com/dotnet/corefx/issues/36618

I've run them again. (you can use the azp command): https://github.com/dotnet/corefx/blob/34118b74d46129b3095e1b4dbad8441c4db62e33/Documentation/project-docs/pullrequest-builds.md#azure-pipelines-commands

@eanova
Copy link
Author

eanova commented Apr 6, 2019

Thanks @ahsonkhan , it's good to know these kind of test errors can occur between different builds and to treat it like a possible issue.

Copy link
Member

@stephentoub stephentoub left a comment

Choose a reason for hiding this comment

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

Thanks!

@stephentoub stephentoub merged commit e5fddbc into dotnet:master Apr 7, 2019
@eanova eanova deleted the BrotliMOveInterop branch April 8, 2019 04:49
@karelz karelz added this to the 3.0 milestone May 22, 2019
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
…fx#36614)

* MovedInteropLocationForBrotli

* Combined Brotli Files


Commit migrated from dotnet/corefx@e5fddbc
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.

4 participants