Skip to content

[workspace] Upgrade gz_math_internal to latest release 8.2.0 #23143

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

Merged
merged 2 commits into from
Jun 28, 2025

Conversation

Aiden2244
Copy link
Contributor

@Aiden2244 Aiden2244 commented Jun 27, 2025

Towards #23138

Reason for exclusion: fails locally.

ERROR: Traceback (most recent call last):
	File "/home/local/KHQ/aiden.mccormack/.cache/bazel/_bazel_aiden.mccormack/d8f67396fa92625d9dd7f1218b66f6d2/external/+internal_repositories+gz_math_internal/BUILD.bazel", line 133, column 24, in <toplevel>
		check_lists_consistency(
	File "/home/local/KHQ/aiden.mccormack/drake-aiden2244/drake/tools/workspace/check_lists_consistency.bzl", line 24, column 13, in check_lists_consistency
		fail("The following files matched a glob of upstream sources, but " +
Error in fail: The following files matched a glob of upstream sources, but were not covered by the package.BUILD.bazel file: ["include/gz/math/AxisAlignedBoxHelpers.hh", "include/gz/math/detail/AxisAlignedBoxHelpers.hh"]
ERROR: /home/local/KHQ/aiden.mccormack/drake-aiden2244/drake/bindings/pydrake/multibody/BUILD.bazel:557:23: in genquery rule //bindings/pydrake/multibody:install_lint_genquery_install: errors were encountered while computing transitive closure of the scope: no such target '@@+internal_repositories+gz_math_internal//:install': target 'install' not declared in package '' defined by /home/local/KHQ/aiden.mccormack/.cache/bazel/_bazel_aiden.mccormack/d8f67396fa92625d9dd7f1218b66f6d2/external/+internal_repositories+gz_math_internal/BUILD.bazel
ERROR: /home/local/KHQ/aiden.mccormack/drake-aiden2244/drake/bindings/pydrake/multibody/BUILD.bazel:557:23: Analysis of target '//bindings/pydrake/multibody:install_lint_genquery_install' failed
ERROR: Analysis of target '//bindings/pydrake/multibody:py/install_lint' failed; build aborted: Analysis failed
INFO: Elapsed time: 0.989s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Fetching some target dependencies failed with errors: Analysis of target '//bindings/pydrake/multibody:py/install_lint' failed; build aborted: in genquery rule //bindings/pydrake/multibody:install_lint_genquery_install: errors were encountered while computing transitive closure of the scope: no such target '@@+internal_repositories+gz_math_internal//:install': target 'install' not declared in package '' defined by /home/local/KHQ/aiden.mccormack/.cache/bazel/_bazel_aiden.mccormack/d8f67396fa92625d9dd7f1218b66f6d2/external/+internal_repositories+gz_math_internal/BUILD.bazel

This change is Reviewable

@Aiden2244 Aiden2244 mentioned this pull request Jun 27, 2025
6 tasks
@jwnimmer-tri
Copy link
Collaborator

+@rpoyner-tri for pushing fixes, please.

@jwnimmer-tri jwnimmer-tri changed the title [workspace] Upgrade gz_math_internal to latest release gz-math8_8.2.0 [workspace] Upgrade gz_math_internal to latest release 8.2.0 Jun 27, 2025
@jwnimmer-tri jwnimmer-tri added the release notes: fix This pull request contains fixes (no new features) label Jun 27, 2025
@jwnimmer-tri jwnimmer-tri self-assigned this Jun 27, 2025
@jwnimmer-tri jwnimmer-tri added the status: single reviewer ok https://drake.mit.edu/reviewable.html label Jun 27, 2025
Copy link
Collaborator

@jwnimmer-tri jwnimmer-tri left a comment

Choose a reason for hiding this comment

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

:lgtm: both pending CI.

Reviewed 1 of 1 files at r1, 1 of 1 files at r2, all commit messages.
Reviewable status: LGTM missing from assignee rpoyner-tri(platform), commits need curation (https://drake.mit.edu/reviewable.html#curated-commits) (waiting on @Aiden2244)

@rpoyner-tri rpoyner-tri added the status: squashing now https://drake.mit.edu/reviewable.html#curated-commits label Jun 28, 2025
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.

+(status: squashing now)

Reviewable status: LGTM missing from assignee rpoyner-tri(platform)

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.

:lgtm:

Reviewable status: :shipit: complete! all discussions resolved, LGTM from assignees rpoyner-tri(platform),jwnimmer-tri(platform)

@rpoyner-tri rpoyner-tri merged commit a972ded into RobotLocomotion:master Jun 28, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release notes: fix This pull request contains fixes (no new features) status: single reviewer ok https://drake.mit.edu/reviewable.html status: squashing now https://drake.mit.edu/reviewable.html#curated-commits
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants