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

Linux build issues #10

Open
ChristophHaag opened this issue Sep 29, 2016 · 5 comments
Open

Linux build issues #10

ChristophHaag opened this issue Sep 29, 2016 · 5 comments

Comments

@ChristophHaag
Copy link

I'll just put everything in one issue instead of making multiple across all the related repositories.

pd-hs:

Two ifdefs need a linux branch (_WIN32 is okay for now)
https://github.com/lukexi/pd-hs/blob/3a17e75cb7a5d20814d4c38e2b98a53e0acc012e/cbits/libpd_openal.c#L1
https://github.com/lukexi/pd-hs/blob/3a17e75cb7a5d20814d4c38e2b98a53e0acc012e/cbits/libpd_openal.c#L172

openvr-hs:

This whole file:
https://github.com/lukexi/openvr-hs/blob/ebbeb8c5393e7bbe8aaa46642fce9776c3af0265/cbits/openvr_mingw.h
OpenVR headers are actually broken upstream. Here are two pull requests that I think are enough to make openvr_capi.h work.
ValveSoftware/openvr#58
ValveSoftware/openvr#278
Not sure how to best do this, but I kinda "maintain" it on https://github.com/ChristophHaag/openvr/blob/master/headers/openvr_capi.h
I haven't checked how much different the mingw version is, maybe needs some merging.

vr-pal:

Windows only libs:
https://github.com/lukexi/vr-pal/blob/c84ecd001e2af4fb73c27d76ffb0b8a4cfd51e47/vr-pal.cabal#L44
https://github.com/lukexi/vr-pal/blob/c84ecd001e2af4fb73c27d76ffb0b8a4cfd51e47/vr-pal.cabal#L89
Windows only code:
https://github.com/lukexi/vr-pal/blob/c84ecd001e2af4fb73c27d76ffb0b8a4cfd51e47/cbits/win32_SetProcessDpiAware.c

This last issue is where I have stopped for now, because it needs some more involved work to remove it from the project. At least that's what I think.

@lukexi
Copy link
Owner

lukexi commented Oct 1, 2016

Thanks so much Christoph! Will get to these asap!

@slowriot
Copy link

PR 58 finally got accepted after only 9 months, so we're making some progress ;)

@ChristophHaag
Copy link
Author

Yea, the stdbool.h one too. To be fair, reviewing such complex pull requests can take a long time!

On a more serious notes, it probably has something to do with Valve demonstrating SteamVR on Linux recently. Looks like things are slowly but surely starting to move.

@slowriot
Copy link

Let's hope so!

@lukexi
Copy link
Owner

lukexi commented Nov 12, 2016

I just finished getting Rumpus back up and running on OS X so Linux shouldn't be too far away now! I need to set up a dev machine/VirtualBox first...

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

3 participants