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

add macOS define #323

Merged
merged 2 commits into from
Aug 5, 2022
Merged

add macOS define #323

merged 2 commits into from
Aug 5, 2022

Conversation

kcking
Copy link
Contributor

@kcking kcking commented Jul 27, 2022

This fixes building the loader on macOS.

With this commit I am able to build the loader on macOS as part of the openxr rust crate. I also have a fork of hotham-simulator working locally, allowing complete (albeit basic) openXR development locally on my macbook.

This fixes building the loader on macOS.

With this commit I am able to build the loader on macOS as part of the [openxr
rust crate](https://github.com/Ralith/openxrs). I have a fork of
[hotham-simulator](https://github.com/leetvr/hotham/wiki/Adding-the-Hotham-Simulator-to-your-development-environment)
working locally, allowing complete (albeit basic) openXR development locally on
my macbook.
@CLAassistant
Copy link

CLAassistant commented Jul 27, 2022

CLA assistant check
All committers have signed the CLA.

@rpavlik
Copy link
Contributor

rpavlik commented Jul 27, 2022

Great, thanks! Can you make a changelog fragment changes/sdk/pr.323.gh.OpenXR-SDK-Source.md too? The readme there has more advice.

@rpavlik rpavlik added the needs changelog fragment Someone, preferably the submitter, needs to add a changelog fragment describing this. label Jul 27, 2022
@kcking
Copy link
Contributor Author

kcking commented Jul 27, 2022

Great, thanks! Can you make a changelog fragment changes/sdk/pr.323.gh.OpenXR-SDK-Source.md too? The readme there has more advice.

Just added a changelog fragment (hope I did it correctly).

@kcking
Copy link
Contributor Author

kcking commented Jul 30, 2022

Friendly bump @rpavlik, thanks again for taking a look!

@rpavlik
Copy link
Contributor

rpavlik commented Jul 30, 2022

Yeah I have to figure out what's wrong with CI, I'll get to it when I'm in the office again.

@rpavlik
Copy link
Contributor

rpavlik commented Aug 4, 2022

It looks like you might somehow have a trailing space on that filename for your changelog file? I didn't know that was possible but that's what it looks like.

@kcking
Copy link
Contributor Author

kcking commented Aug 5, 2022

It looks like you might somehow have a trailing space on that filename for your changelog file? I didn't know that was possible but that's what it looks like.

Wow you're right, I had no idea that was possible either 😅 Looks like it's fixed now

@rpavlik rpavlik merged commit 98f804e into KhronosGroup:main Aug 5, 2022
kcking added a commit to kcking/openxrs that referenced this pull request Oct 4, 2022
Ralith pushed a commit to Ralith/openxrs that referenced this pull request Oct 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs changelog fragment Someone, preferably the submitter, needs to add a changelog fragment describing this.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants