Skip to content

Conversation

@dotnet-maestro
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Jul 13, 2023

This pull request updates the following dependencies

From https://github.com/dotnet/arcade

  • Subscription: 840bf684-917a-4879-f62c-08d8e93a0c51
  • Build: 20230809.5
  • Date Produced: August 9, 2023 3:51:11 PM UTC
  • Commit: bebd2de1988036b47ecefc7e72b66ce2f403e43d
  • Branch: refs/heads/main
Microsoft Reviewers: Open in CodeFlow

…712.5

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.23361.1 -> To Version 8.0.0-beta.23362.5
@dotnet-maestro dotnet-maestro bot requested a review from a team as a code owner July 13, 2023 12:21
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Go, you big red fire engine!

@ghost ghost added the PR metadata: Label to tag PRs, to facilitate with triage label Jul 13, 2023
@ghost ghost requested review from dipeshmsft and singhashish-wpf July 13, 2023 12:21
@singhashish-wpf
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

No commit pushedDate could be found for PR 8030 in repo dotnet/wpf

…713.3

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.23361.1 -> To Version 8.0.0-beta.23363.3
@mmitche
Copy link
Member

mmitche commented Jul 14, 2023

This has something to do with sourcelink moving into the SDK.

mmitche and others added 2 commits July 14, 2023 07:36
…714.2

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.23361.1 -> To Version 8.0.0-beta.23364.2
@dipeshmsft
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@lewing
Copy link
Member

lewing commented Jul 18, 2023

cc @ViktorHofer

@ViktorHofer
Copy link
Member

ViktorHofer commented Jul 18, 2023

cc @ViktorHofer

Thanks for the ping. We have been discussing this for the last days as it isn't exactly clear how to respond to vcxproj support. We believe we found an option and will now work on a fix.

dotnet-maestro bot added 3 commits July 20, 2023 12:12
…719.2

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.23361.1 -> To Version 8.0.0-beta.23369.2
…720.6

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.23361.1 -> To Version 8.0.0-beta.23370.6
…721.1

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.23361.1 -> To Version 8.0.0-beta.23371.1
@singhashish-wpf
Copy link
Contributor

@ViktorHofer Do we have a way to address this failure? Please inform if there is something required from wpf side.

@ViktorHofer
Copy link
Member

@tmat @mmitche I'm out this week and couldn't complete a fix (as we discussed offline). Would you be able to help out here? Otherwise I can revisit this when I'm back, next week.

We could still consider adding the package references directly just here or in Arcade.Sdk.

dotnet-maestro bot added 3 commits July 27, 2023 12:12
…726.1

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.23361.1 -> To Version 8.0.0-beta.23376.1
…727.1

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.23361.1 -> To Version 8.0.0-beta.23377.1
…728.2

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.23361.1 -> To Version 8.0.0-beta.23378.2
@singhashish-wpf
Copy link
Contributor

@ViktorHofer @mmitche Is there something which is to be done from wpf side? This has been blocked for a week now.

@ViktorHofer
Copy link
Member

ViktorHofer commented Jul 31, 2023

I'm back this week and will continue my investigation.

…731.1

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.23361.1 -> To Version 8.0.0-beta.23381.1
…801.3

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.23361.1 -> To Version 8.0.0-beta.23401.3
…802.2

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.23361.1 -> To Version 8.0.0-beta.23402.2
…803.7

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.23361.1 -> To Version 8.0.0-beta.23403.7
…804.2

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.23361.1 -> To Version 8.0.0-beta.23404.2
@singhashish-wpf
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ViktorHofer
Copy link
Member

ViktorHofer commented Aug 7, 2023

We are still in the same state and I had to shift to something high prio last week. Will chat with @mmitche about this.

dotnet-maestro bot and others added 6 commits August 8, 2023 12:11
…807.1

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.23361.1 -> To Version 8.0.0-beta.23407.1
…808.6

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.23361.1 -> To Version 8.0.0-beta.23408.6
…809.5

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.23361.1 -> To Version 8.0.0-beta.23409.5
Sourcelink support was removed from arcade and is now provided by the .NET SDK. Since the .NET sdk targets aren't automatically brought in, add appropriate package references when building vcxproj.
@mmitche
Copy link
Member

mmitche commented Aug 10, 2023

I think I've got this resolved. PTAL

@dotnet-maestro dotnet-maestro bot merged commit 6f41c13 into main Aug 10, 2023
@dotnet-maestro dotnet-maestro bot deleted the darc-main-044f1e8c-cb21-4105-8301-6c61cb960ec3 branch August 10, 2023 22:46
@mmitche
Copy link
Member

mmitche commented Aug 11, 2023

Well, Maestro just up and merged that. Feel free to review and we can change if need be.

@@ -1,2 +1,2 @@
@echo off
powershell -ExecutionPolicy ByPass -NoProfile -command "& """%~dp0Build.ps1""" -restore -build -sign -pack -publish -ci %*"
powershell -ExecutionPolicy ByPass -NoProfile -command "& """%~dp0Build.ps1""" -restore -build -test -sign -pack -publish -ci %*"
Copy link
Contributor

Choose a reason for hiding this comment

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

It looks like this PR reverted some changes from #8013.

Copy link
Member

Choose a reason for hiding this comment

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

eng/common/* is 'controlled' by the arcade repo and shouldn't be checked into. Any changes there would be overwritten on the next arcade update. The right way to do that is to make an eng/cibuild.cmd which calls build.ps1 the way you want and use that in CI scripts.

Copy link
Contributor

Choose a reason for hiding this comment

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

@ThomasGoulet73 - We've moved the scripts here - #8036

Copy link
Contributor

Choose a reason for hiding this comment

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

@pchaurasia14 Alright thanks, just wanted to make sure it didn't break anything since I remembered seeing manual changes to these files.

@ViktorHofer
Copy link
Member

Thanks a lot Matt 👍

@ghost ghost locked as resolved and limited conversation to collaborators Sep 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

PR metadata: Label to tag PRs, to facilitate with triage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants