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

Rewrite build fails on GitHub actions #98

Closed
ThreeDeeJay opened this issue Dec 20, 2023 · 4 comments
Closed

Rewrite build fails on GitHub actions #98

ThreeDeeJay opened this issue Dec 20, 2023 · 4 comments

Comments

@ThreeDeeJay
Copy link

I cloned the c++ rewrite branch to try setting up daily builds like PR #52 but I'm getting compilation errors.
Initially I though it was because the project was targeting Visual Studio 2017 which works on Appveyor, but the master branch also targets VS 2017 and builds fine with VS 2022. I also tried using the oldest runner available on Github actions (VS 2019), but it still failed, even with more compilation errors. So any idea why it's failing?

Also, would you recommend using it as a replacement for the master branch? or are there any major features missing?🤔
I'm planning to record and upload a full Prince of Persia: The Two Thrones playthough soon™️, so I figured maybe I could try the rewrite to see if I can spot any issues, so it'd come in handy to know what to look for 👀👌

@kcat
Copy link
Owner

kcat commented Dec 20, 2023

The compilation errors seem to be from a missing <string> include in propset.cpp.

DSSCL_WRITEPRIMARY, Capture, and FullDuplex aren't working/implemented currently. Some of the DirectSoundPrivate interface is also not all there. Many things should work though, so it's definitely worth trying out.

@UserGit-2021
Copy link

The compilation errors seem to be from a missing <string> include in propset.cpp.

DSSCL_WRITEPRIMARY, Capture, and FullDuplex aren't working/implemented currently. Some of the DirectSoundPrivate interface is also not all there. Many things should work though, so it's definitely worth trying out.

Should we expect a DSOAL update in the near future? There has been no update for a long time, 10 months have passed.

@mirh
Copy link

mirh commented Jan 9, 2024

Atm work is flying like there's no tomorrow on openal-soft
So... delighted as I may also be to see this getting progress, let's not disturb the brisk development focus

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

No branches or pull requests

4 participants