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

Expand Linux disk for 3p package environments #278

Merged
merged 4 commits into from
Mar 5, 2025

Conversation

amzn-changml
Copy link
Contributor

Adds a disk resizing workflow to the packaging environment. This will increase the root disk size to 20GB and remove non-essential packages

Tested in my own branch with the OpenXR package: https://github.com/amzn-changml/3p-package-source/actions/runs/13618717178

amzn-changml and others added 3 commits March 2, 2025 18:31
This update makes it straightforward to build O3DE applications for Meta
Quest devices, because the developer is not required anymore to download
and decompress the `Oculus OpenXR Mobile SDK`.

To understand this change, let's review a little bit of history: When
making O3DE apps for Meta Quest devices, the developer was instructed to
manually download and decompress the
`Oculus OpenXR Mobile SDK` into a particular directory of the `OpenXRVk`
Gem.

Those instructions worked well until SDK v62. Starting SDK v63+ (It's
currently at v72) Meta stopped distributing pre-compiled OpenXR Loader
libraries, and now applications are simply required to get the OpenXR
Loader directly from github: https://github.com/KhronosGroup/OpenXR-SDK

In particular, this PR picks version 1.1.41 because that's the version
used by `v72` of `Oculus OpenXR Mobile SDK`.

In other words, with this change, OpenXR is now supported in its purest
form, because there are no custom steps for a particular vendor.

Also, it was found that when creating the Android package, the
Android-tool.cmake tool doesn't copy the headers in the "install"
directories. So I added an improvement to the
`pull_and_build_from_git.py` script, where the `"extra_files_to_copy"`
property now supports copying directories, with a third, optional,
parameter, which is a list of file patterns to ignore when deepcopying
the directories. Example:
```json
{
    "extra_files_to_copy": [
      ["temp/src/include", "OpenXR/include", ["*.txt", "*.license"]]
    ],
}
```
Additionally, when copying files, it now supports automatic creation of
non-existing sub-folders.

---------

Signed-off-by: galibzon <[email protected]>
@amzn-changml amzn-changml requested review from spham-amzn and a team March 3, 2025 02:43
@amzn-changml amzn-changml merged commit 5eb11ba into o3de:main Mar 5, 2025
1 check passed
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

Successfully merging this pull request may close these issues.

3 participants