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

Use live M.Bcl.AsyncInterfaces dependency #94451

Merged
merged 2 commits into from
Nov 7, 2023
Merged

Conversation

ViktorHofer
Copy link
Member

@ViktorHofer ViktorHofer commented Nov 7, 2023

Microsoft.Bcl.AsyncInterfaces is still built in the main branch and shouldn't be brought in as a prebuilt by the TimeProvider project.

@ViktorHofer ViktorHofer self-assigned this Nov 7, 2023
@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Nov 7, 2023
Copy link
Member

@ericstj ericstj left a comment

Choose a reason for hiding this comment

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

LGTM. I assume we need to service this as well? Maybe just as infrastructure so that the package is set up to use latest just in case it needs to ship in the future.

@ViktorHofer
Copy link
Member Author

ViktorHofer commented Nov 7, 2023

Yes, we should backport this into release/8.0. I would even go as far as releasing a 8.0.1 version of Microsoft.Bcl.TimeProvider.

@ViktorHofer
Copy link
Member Author

/backport to release/8.0-staging

Copy link
Contributor

github-actions bot commented Nov 7, 2023

Started backporting to release/8.0-staging: https://github.com/dotnet/runtime/actions/runs/6784746659

Copy link
Contributor

github-actions bot commented Nov 7, 2023

@ViktorHofer backporting to release/8.0-staging failed, the patch most likely resulted in conflicts:

$ git am --3way --ignore-whitespace --keep-non-patch changes.patch

Applying: Use live M.Bcl.AsyncInterfaces dependency
Using index info to reconstruct a base tree...
M	src/libraries/Microsoft.Bcl.TimeProvider/ref/Microsoft.Bcl.TimeProvider.csproj
Falling back to patching base and 3-way merge...
Auto-merging src/libraries/Microsoft.Bcl.TimeProvider/ref/Microsoft.Bcl.TimeProvider.csproj
CONFLICT (content): Merge conflict in src/libraries/Microsoft.Bcl.TimeProvider/ref/Microsoft.Bcl.TimeProvider.csproj
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Use live M.Bcl.AsyncInterfaces dependency
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
Error: The process '/usr/bin/git' failed with exit code 128

Please backport manually!

Copy link
Contributor

github-actions bot commented Nov 7, 2023

@ViktorHofer an error occurred while backporting to release/8.0-staging, please check the run log for details!

Error: git am failed, most likely due to a merge conflict.

@ViktorHofer
Copy link
Member Author

Here's the manual backport: #94459

@ViktorHofer ViktorHofer added area-System.DateTime and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Nov 7, 2023
@ViktorHofer ViktorHofer added this to the 9.0.0 milestone Nov 7, 2023
@ghost
Copy link

ghost commented Nov 7, 2023

Tagging subscribers to this area: @dotnet/area-system-datetime
See info in area-owners.md if you want to be subscribed.

Issue Details

Microsoft.Bcl.AsyncInterfaces is still built in the main branch and shouldn't be brought in as a prebuilt by the TimeProvider project.

Author: ViktorHofer
Assignees: ViktorHofer
Labels:

area-System.DateTime

Milestone: -

@ViktorHofer ViktorHofer merged commit 4feec00 into main Nov 7, 2023
106 of 109 checks passed
@ViktorHofer ViktorHofer deleted the ViktorHofer-patch-1 branch November 7, 2023 17:45
carlossanlop pushed a commit that referenced this pull request Nov 21, 2023
* [release/8.0-staging] Use live M.Bcl.AsyncInterfaces dependency

Manual backport of #94451

* Update Microsoft.Bcl.TimeProvider.csproj

* Update Microsoft.Bcl.TimeProvider.csproj

* Update Microsoft.Bcl.TimeProvider.csproj
@ghost ghost locked as resolved and limited conversation to collaborators Dec 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants