Skip to content

Conversation

footballhead
Copy link
Collaborator

@footballhead footballhead commented Aug 21, 2025

Last subtree update, we got two new Android targets:

  • multi-win-replay
  • quest_replay

As a result, build times doubled. In order to reduce build times, only compile parts of GFXR that we use. This also means that we don't compile the test apps

@footballhead
Copy link
Collaborator Author

footballhead commented Aug 21, 2025

Gradle finished but there were CMake errors.

-- Move gfxr files
CMake Error: Problem with archive_read_open_file(): Failed to open '/home/runner/work/dive/dive/third_party/gfxreconstruct/android/layer/build/outputs/aar/layer-release.aar'
CMake Error at CMakeLists.txt:118 (file):
  file failed to extract:
  /home/runner/work/dive/dive/third_party/gfxreconstruct/android/layer/build/outputs/aar/layer-release.aar


-- Configuring incomplete, errors occurred!

EDIT: Fixed. Needed to build both layer and release explicitly

Last subtree update, we got two new Android targets:

- multi-win-replay
- quest_replay

As a result, build times doubled. In order to reduce build times, only compile parts of GFXR that we use. Since replay depends on the capture layer, asking Gradle to build replay will cause both to be built
@footballhead footballhead force-pushed the build_android_gfxr_replay_only branch from 420001b to 1825883 Compare September 18, 2025 14:37
replay's dependency on layer will only build AAR in debug. In release,
need to build both. Fortunately, building both can be accomplished with
one command.
@footballhead footballhead changed the title build_android: Only build replay (and layer) build_android: Only build replay and layer Sep 24, 2025
@footballhead footballhead marked this pull request as ready for review September 24, 2025 16:29
@footballhead
Copy link
Collaborator Author

Copy link
Collaborator

@angela28chen angela28chen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for making this change!

@footballhead footballhead enabled auto-merge (squash) September 24, 2025 18:05
@footballhead footballhead merged commit 870d5d5 into google:main Sep 24, 2025
9 checks passed
@footballhead footballhead deleted the build_android_gfxr_replay_only branch September 24, 2025 21:24
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