Skip to content

enable ABI compatibility action check. - #3262

Merged
fujitatomoya merged 3 commits into
rollingfrom
fujitatomoya/enable-abi-check-action
Sep 11, 2026
Merged

enable ABI compatibility action check.#3262
fujitatomoya merged 3 commits into
rollingfrom
fujitatomoya/enable-abi-check-action

Conversation

@fujitatomoya

@fujitatomoya fujitatomoya commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

Description

trial for #2555

Fixes # (issue)

Is this user-facing behavior change?

No

Did you use Generative AI?

No

Additional Information

This needs to be test in flight for a while to make sure the action can detect ABI compatibility including backports.
After this PR is merged, i will have to create some test PRs to see the action works as designed in this repository.

Signed-off-by: Tomoya.Fujita <fujita.tomoya@triorb.co.jp>
@fujitatomoya fujitatomoya self-assigned this Sep 10, 2026

@wjwwood wjwwood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I like it. +1 from me, with green CI

…nges.

Signed-off-by: Tomoya.Fujita <fujita.tomoya@triorb.co.jp>
@github-actions

github-actions Bot commented Sep 11, 2026

Copy link
Copy Markdown

ABI Compliance Check

Verdict: compatible

No ABI changes detected.

Compared:

  • Base: lib-base/librclcpp_action.so
  • Head: lib-pr/librclcpp_action.so @ 74309d6
Full abidiff report
(empty report — no differences printed by abidiff)

Updated for commit 74309d6 · suppressions: /home/runner/work/_temp/ros2-abi-suppressions.txt

@github-actions

github-actions Bot commented Sep 11, 2026

Copy link
Copy Markdown

ABI Compliance Check

Verdict: compatible

No ABI changes detected.

Compared:

  • Base: lib-base/libcomponent_manager.so
  • Head: lib-pr/libcomponent_manager.so @ 74309d6
Full abidiff report
(empty report — no differences printed by abidiff)

Updated for commit 74309d6 · suppressions: /home/runner/work/_temp/ros2-abi-suppressions.txt

@github-actions

github-actions Bot commented Sep 11, 2026

Copy link
Copy Markdown

ABI Compliance Check

Verdict: compatible

No ABI changes detected.

Compared:

  • Base: lib-base/librclcpp_lifecycle.so
  • Head: lib-pr/librclcpp_lifecycle.so @ 74309d6
Full abidiff report
(empty report — no differences printed by abidiff)

Updated for commit 74309d6 · suppressions: /home/runner/work/_temp/ros2-abi-suppressions.txt

@github-actions

github-actions Bot commented Sep 11, 2026

Copy link
Copy Markdown

ABI Compliance Check

Verdict: compatible

No ABI changes detected.

Compared:

  • Base: lib-base/librclcpp.so
  • Head: lib-pr/librclcpp.so @ 74309d6
Full abidiff report
(empty report — no differences printed by abidiff)

Updated for commit 74309d6 · suppressions: /home/runner/work/_temp/ros2-abi-suppressions.txt

@fujitatomoya

Copy link
Copy Markdown
Collaborator Author

yup, that seems to be working. supporting ROS 2 core development is more complicated than expected.
the following features are added in https://github.com/fujitatomoya/ros2-abi-action.

  • multiple libraries and packages in the single repository.
  • source code build is always required, dependent libraries are not released yet.
  • there could be bundled PRs, those also need to built with.

Comment thread .github/workflows/abi.yaml
Comment thread .github/workflows/abi.yaml
Comment thread .github/workflows/abi.yaml
Signed-off-by: Tomoya.Fujita <fujita.tomoya@triorb.co.jp>
@fujitatomoya
fujitatomoya merged commit e47c084 into rolling Sep 11, 2026
10 checks passed
@christophebedard
christophebedard deleted the fujitatomoya/enable-abi-check-action branch September 11, 2026 19:24
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.

4 participants