Skip to content

[workspace] Upgrade openusd_internal to latest v25.05 #22969

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

Conversation

Aiden2244
Copy link
Contributor

@Aiden2244 Aiden2244 commented May 5, 2025

Towards #22928

Sample Console Log:

[5:30:08 PM]  ERROR: /media/ephemeral0/ubuntu/workspace/linux-jammy-clang-bazel-experimental-everything-release/_bazel_ubuntu/5e0ced017c744e846b192696bb0e77f4/external/+internal_repositories+openusd_internal/BUILD.bazel:28:21: Action external/+internal_repositories+openusd_internal/pxr/pxr.h failed: (Exit 1): cmake_configure_file failed: error executing Action command (from target @@+internal_repositories+openusd_internal//:config_genrule) bazel-out/k8-opt-exec-ST-d57f47055a04/bin/tools/workspace/cmake_configure_file --input external/+internal_repositories+openusd_internal/pxr/pxr.h.in --output ... (remaining 10 arguments skipped)
[5:30:08 PM]  
[5:30:08 PM]  Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
[5:30:08 PM]  Traceback (most recent call last):
[5:30:08 PM]    File "/media/ephemeral0/ubuntu/workspace/linux-jammy-clang-bazel-experimental-everything-release/_bazel_ubuntu/5e0ced017c744e846b192696bb0e77f4/sandbox/linux-sandbox/4783/execroot/_main/bazel-out/k8-opt-exec-ST-d57f47055a04/bin/tools/workspace/cmake_configure_file.runfiles/_main/tools/workspace/cmake_configure_file.py", line 247, in <module>
[5:30:08 PM]      main()
[5:30:08 PM]    File "/media/ephemeral0/ubuntu/workspace/linux-jammy-clang-bazel-experimental-everything-release/_bazel_ubuntu/5e0ced017c744e846b192696bb0e77f4/sandbox/linux-sandbox/4783/execroot/_main/bazel-out/k8-opt-exec-ST-d57f47055a04/bin/tools/workspace/cmake_configure_file.runfiles/_main/tools/workspace/cmake_configure_file.py", line 239, in main
[5:30:08 PM]      raise RuntimeError(f"The definitions of {sorted(unused_vars)} were"
[5:30:08 PM]  RuntimeError: The definitions of ['PXR_USE_INTERNAL_BOOST_PYTHON'] were ignored and therefore seem like dead code; remove them from defines= or undefines=.

Sample Jenkins Job: https://drake-jenkins.csail.mit.edu/job/linux-jammy-clang-bazel-experimental-everything-release/8279/consoleFull


This change is Reviewable

@jwnimmer-tri

This comment was marked as resolved.

@Aiden2244 Aiden2244 force-pushed the 22928/abseil-cpp-internal-upgrade branch from 1442c88 to 884b7ad Compare May 5, 2025 15:58
@Aiden2244 Aiden2244 changed the title [workspace] Upgrade abseil_cpp_internal to latest [workspace] Upgrade openusd_internal to latest v25.05 May 5, 2025
Aiden2244

This comment was marked as resolved.

@jwnimmer-tri
Copy link
Collaborator

+@rpoyner-tri I'll volunteer you to own this, please (pushing build fixes to the branch). Extremely low priority. Let's say the goal is to finish it before the next upgrade cycle on June 1st.

Copy link
Contributor

@rpoyner-tri rpoyner-tri left a comment

Choose a reason for hiding this comment

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

Reviewed 2 of 3 files at r2.
Reviewable status: LGTM missing from assignee rpoyner-tri(platform), needs at least two assigned reviewers, missing label for release notes (waiting on @Aiden2244)

@jwnimmer-tri
Copy link
Collaborator

BTW probably worth going up the minor patch to https://github.com/PixarAnimationStudios/OpenUSD/releases/tag/v25.05.01

@jwnimmer-tri
Copy link
Collaborator

(Or in the alternative, now might be the time to declare the USD parser as abandoned, and revert it all until someone gets interested in it again.)

@Aiden2244 Aiden2244 mentioned this pull request Jun 3, 2025
10 tasks
Copy link
Contributor

@rpoyner-tri rpoyner-tri left a comment

Choose a reason for hiding this comment

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

I recommend closing this, in favor of #23065 .

Reviewable status: LGTM missing from assignee rpoyner-tri(platform), needs at least two assigned reviewers, missing label for release notes (waiting on @Aiden2244)

@rpoyner-tri rpoyner-tri closed this Jun 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants