Skip to content

Disable schema test if xmllint is too new - #1702

Open
scpeters wants to merge 1 commit into
sdf16from
scpeters/check_xmllint_version_for_schema_test
Open

Disable schema test if xmllint is too new#1702
scpeters wants to merge 1 commit into
sdf16from
scpeters/check_xmllint_version_for_schema_test

Conversation

@scpeters

Copy link
Copy Markdown
Member

🦟 Bug fix

Mitigation for #1656, manual backport of #1658 and part of #1655

Summary

As noted in #1656, INTEGRATION_schema_test fails with versions of xmllint newer than 2.15. This disables the test if the xmllint version is too new. It also fixes the warning when xmllint is not found.

Backport Policy

  • This is safe to backport to the following versions:
    • Jetty
    • Ionic
    • Harmonic
    • Fortress
  • This should not be backported
  • I am not sure
  • Other (fill in yourself)

Checklist

  • Signed all commits for DCO
  • Added a screen capture or video to the PR description that demonstrates the fix (as needed)
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • Updated Bazel files (if adding new files). Created an issue otherwise.
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers
  • Was GenAI used to generate this PR? If so, make sure to add "Assisted-by" to your commits. (See this policy for more info.)

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by and Generated-by messages.

Backports: If this is a backport, please use Rebase and Merge instead.

* Warn when xmllint is not found or version is not detected
* Status message if schema test is skipped

Signed-off-by: Steve Peters <scpeters@intrinsic.ai>
@github-actions github-actions Bot added the 🪵 jetty Gazebo Jetty label Aug 14, 2026
@scpeters
scpeters marked this pull request as ready for review August 14, 2026 04:09
@scpeters
scpeters requested a review from azeey as a code owner August 14, 2026 04:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🪵 jetty Gazebo Jetty

Projects

Status: Inbox

Development

Successfully merging this pull request may close these issues.

2 participants