Skip to content

Conversation

JohnMcPMS
Copy link
Member

@JohnMcPMS JohnMcPMS commented May 8, 2025

Change

Even if not found, attempt to use DSCv3 resources. This is largely to avoid PowerShell/DSC#786, which only seems to affect dsc.exe's ability to run processes when listing adapted resources.

Also applies the DiagnosticTraceEnabled state to the elevated processor. This is set when the config logging area is enabled and set to verbose.

Validation

When running manually, the logs indicate that the adapted list invocations are failing, but the attempt to use the resource still succeeds.

Microsoft Reviewers: Open in CodeFlow

@JohnMcPMS JohnMcPMS requested a review from a team as a code owner May 8, 2025 21:59
yao-msft
yao-msft previously approved these changes May 8, 2025
Copy link
Contributor

@yao-msft yao-msft left a comment

Choose a reason for hiding this comment

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

Should we set that Microsoft.Windows.Settings elevation to true since this change will mitigate the issue?

@JohnMcPMS JohnMcPMS merged commit 11c0786 into microsoft:master May 9, 2025
9 checks passed
@JohnMcPMS JohnMcPMS deleted the run-anyway branch May 9, 2025 16:49
yao-msft added a commit to yao-msft/winget-cli that referenced this pull request Jun 6, 2025
yao-msft added a commit to yao-msft/winget-cli that referenced this pull request Jun 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants