This repository was archived by the owner on Jun 5, 2025. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 20
Enable the ability to toggle App Verifier in Image File Execution Options (IFEO) for the target app. #3840
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
No service, No Elevation! remove stale comment initial plug at a separate tool exe New tool app creation checkpoint. more telemetry more telemetry, and basic arg parsing per-targetapp instancing remove todo
jaholme
reviewed
Sep 11, 2024
tools/Utilities/IfeoTool/Assets/IfeoTool/LargeTile.scale-100.png
Outdated
Show resolved
Hide resolved
jaholme
reviewed
Sep 11, 2024
tools/DevDiagnostics/DevHome.DevDiagnostics/Helpers/ImageOptionsInternalTool.cs
Outdated
Show resolved
Hide resolved
tools/DevDiagnostics/DevHome.DevDiagnostics/Helpers/ImageOptionsInternalTool.cs
Show resolved
Hide resolved
tools/Utilities/IfeoTool/Assets/IfeoTool/SplashScreen.scale-100.png
Outdated
Show resolved
Hide resolved
timkur
reviewed
Sep 12, 2024
tools/DevDiagnostics/DevHome.DevDiagnostics/Helpers/ImageOptionsInternalTool.cs
Outdated
Show resolved
Hide resolved
timkur
reviewed
Sep 12, 2024
tools/DevDiagnostics/DevHome.DevDiagnostics/Helpers/ImageOptionsInternalTool.cs
Show resolved
Hide resolved
timkur
reviewed
Sep 12, 2024
tools/DevDiagnostics/DevHome.DevDiagnostics/Helpers/ImageOptionsInternalTool.cs
Outdated
Show resolved
Hide resolved
timkur
reviewed
Sep 12, 2024
Contributor
|
Add a screenshot of your new tool in the PR description. Show off your handiwork! :) |
Contributor
Contributor
Author
No idea TBH, as I only ever used either VS, VSCode, or Git for all of my operations. Any suggestions on what to look for to identify, fix, and prevent? Seems like a common issue.... |
jaholme
approved these changes
Sep 12, 2024
Contributor
jaholme
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
![]()
andreww-msft
approved these changes
Sep 12, 2024
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary of the pull request
Add's a new internal tool on the bar for enabling AppVerifier, and possibly other Image File Execution Options (IFEO) in the future.
Detailed description of the pull request / Additional comments
Provided a Window that runs elevated with toggle switches to enable/disable options. Currently the UI is the only thing limiting new options being added, and AppVerifier is the only current option included.
The intention is that since AppVerifier produces WER dumps, they (the results) will show up in that section of the UI for viewing purposes.
Validation steps performed
Manual scenario smoke tests
PR checklist