-
Notifications
You must be signed in to change notification settings - Fork 51
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
Comments
The compilation errors seem to be from a missing
|
Should we expect a DSOAL update in the near future? There has been no update for a long time, 10 months have passed. |
Atm work is flying like there's no tomorrow on openal-soft |
Looks like one of the recent commits fixed this because GitHub actions builds of the rewrite branch are magically succeeding now so I'll go ahead and close this issue 👀👌 |
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 👀👌
The text was updated successfully, but these errors were encountered: