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

Enable helloxr building with latest Android Studio canary #393

Merged
merged 3 commits into from
Apr 27, 2023

Conversation

utzcoz
Copy link
Contributor

@utzcoz utzcoz commented Apr 1, 2023

It needs us to upgrade Gradle version.

@utzcoz utzcoz force-pushed the build-helloxr-with-latest-AS-canary branch from df0e7e4 to 068fafd Compare April 1, 2023 05:16
@utzcoz
Copy link
Contributor Author

utzcoz commented Apr 1, 2023

@rpavlik Great news, we can use AGP 7.4.x directly without previous cmake integration problems.

@utzcoz
Copy link
Contributor Author

utzcoz commented Apr 1, 2023

I used command from https://docs.gradle.org/current/userguide/gradle_wrapper.html to update Gradle wrapper files including gradlew and gradlew.bat.

@rpavlik
Copy link
Contributor

rpavlik commented Apr 4, 2023

Excellent, thanks!

@utzcoz utzcoz force-pushed the build-helloxr-with-latest-AS-canary branch 2 times, most recently from 125ffa4 to 3a35eb3 Compare April 5, 2023 07:45
@rpavlik-bot rpavlik-bot added the synced to gitlab Synchronized to OpenXR internal GitLab label Apr 5, 2023
@rpavlik-bot
Copy link
Collaborator

An issue (number 1983) has been filed to correspond to this pull request in the internal Khronos GitLab (Khronos members only: KHR:openxr/openxr#1983 ), to facilitate working group processes.

This GitHub pull request will continue to be the main site of discussion.

Looks like 7.4.2 fixes the cmake integration problem.
Signed-off-by: utzcoz <utzcoz@outlook.com>
Signed-off-by: utzcoz <utzcoz@outlook.com>
Signed-off-by: utzcoz <utzcoz@outlook.com>
@utzcoz utzcoz force-pushed the build-helloxr-with-latest-AS-canary branch from 3a35eb3 to e33bbd2 Compare April 8, 2023 03:07
@utzcoz
Copy link
Contributor Author

utzcoz commented Apr 8, 2023

@rpavlik I think it is ready for reviewing again.

@utzcoz
Copy link
Contributor Author

utzcoz commented Apr 27, 2023

@rpavlik Friendly ping for reviewing.

@rpavlik
Copy link
Contributor

rpavlik commented Apr 27, 2023

Thanks!

@rpavlik rpavlik merged commit 1693690 into KhronosGroup:main Apr 27, 2023
@utzcoz utzcoz deleted the build-helloxr-with-latest-AS-canary branch April 27, 2023 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
synced to gitlab Synchronized to OpenXR internal GitLab
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants