Skip to content

fix new HalfLife alyx release - #426

Merged
Supreeeme merged 3 commits into
Supreeeme:mainfrom
Mr-Zero88:hla
Oct 7, 2026
Merged

Supreeeme merged 3 commits into
Supreeeme:mainfrom
Mr-Zero88:hla

Conversation

@Mr-Zero88

@Mr-Zero88 Mr-Zero88 commented Sep 15, 2026 •

Copy link
Copy Markdown
Contributor

Stubs the GetDXGIOutputInfo and GetD3D9AdapterIndex to return 0.
Proton calls GetD3D9AdapterIndex but ignores its return value so what we return is irelevant. For GetDXGIOutputInfo it does actualy pass though the output but a users has tested it and returning 0 there as well also solves it.

This also adds a interface forward form 16 to 22 for GetOutputDevice.

Updates openxrs to latest git to include Ralith/openxrs#224 fix.

…plemented warnings

Signed-off-by: Mats Huesmann <huesmann.mats@gmail.com>
Signed-off-by: Mats Huesmann <huesmann.mats@gmail.com>
@Mr-Zero88
Mr-Zero88 marked this pull request as ready for review September 15, 2026 16:40
@imbusy

imbusy commented Sep 16, 2026 •

Copy link
Copy Markdown

👍 it worked yesterday at noon Pacific time and then Alyx broke in the evening.

@imbusy

imbusy commented Sep 17, 2026

Copy link
Copy Markdown

Tested the change and it works! Then tested against the latest release and it crashed with not implemented error. So seems to have a positive effect.

@Supreeeme Supreeeme left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Looks like the tests need to be updated?

Comment thread Cargo.toml Outdated
@Mr-Zero88
Mr-Zero88 force-pushed the hla branch 2 times, most recently from 59f0a79 to 250b94e Compare September 24, 2026 04:26
@Mr-Zero88

Copy link
Copy Markdown
Contributor Author

Looks like the tests need to be updated?

Ah, I missed the fakexr entry while adding platform_info. The tests itself should be fine.

Signed-off-by: Mats Huesmann <huesmann.mats@gmail.com>
@Supreeeme
Supreeeme merged commit 1511bbe into Supreeeme:main Oct 7, 2026
3 checks passed
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