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

AtlasEngine: Fix some Windows 10 <14393 bugs #15485

Merged
merged 1 commit into from
Jun 5, 2023

Conversation

lhecker
Copy link
Member

@lhecker lhecker commented May 30, 2023

This fixes a couple spots where I wasn't properly checking
for the existence of some optional D2D interfaces.

Validation Steps Performed

I haven't tested this and don't intend to do it just yet.
Windows Terminal requires build 19041 at least anyways.

@lhecker lhecker added Issue-Bug It either shouldn't be doing this or needs an investigation. Area-AtlasEngine labels May 30, 2023
@carlos-zamora
Copy link
Member

I'm guessing we want to service this?

@lhecker
Copy link
Member Author

lhecker commented May 30, 2023

I'm guessing we want to service this?

No we don't need to. Windows Terminal's minimum build requirement is 19041, so it'll never hit this. It'll be a concern once we ship this to Visual Studio which then ships this to Windows 7/8/old-10.

@DHowett DHowett added the Needs-Second It's a PR that needs another sign-off label Jun 2, 2023
@DHowett DHowett enabled auto-merge (squash) June 2, 2023 18:38
@DHowett DHowett merged commit f0705fb into main Jun 5, 2023
@DHowett DHowett deleted the dev/lhecker/atlas-engine-fixup branch June 5, 2023 23:36
@microsoft-github-policy-service microsoft-github-policy-service bot removed the Needs-Second It's a PR that needs another sign-off label Jun 5, 2023
DHowett pushed a commit that referenced this pull request Jul 27, 2023
This fixes a couple spots where I wasn't properly checking
for the existence of some optional D2D interfaces.

## Validation Steps Performed
I haven't tested this and don't intend to do it just yet.
Windows Terminal requires build 19041 at least anyways.

(cherry picked from commit f0705fb)
Service-Card-Id: 89409006
Service-Version: 1.18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-AtlasEngine Issue-Bug It either shouldn't be doing this or needs an investigation.
Projects
Development

Successfully merging this pull request may close these issues.

3 participants