update minimum swig to 4.4.1#1383
Merged
Merged
Conversation
This provides consistent ABI 5 support across BSK and BSK-SDK plugin builds
a70e547 to
230380a
Compare
schaubh
added a commit
that referenced
this pull request
May 3, 2026
This provides consistent ABI 5 support across BSK and BSK-SDK plugin builds
schaubh
added a commit
that referenced
this pull request
May 3, 2026
schaubh
added a commit
that referenced
this pull request
May 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR raises the supported SWIG 4.x build requirement to SWIG 4.4.1 for Basilisk source builds and Python build requirements. The CMake SWIG detection was simplified to a single versioned
find_package(SWIG 4.4.1...<5 REQUIRED COMPONENTS python)check now that the minimum SWIG version is unconditional.This update provides SWIG ABI 5 support between BSK and BSK plugins.
Verification
Validated locally using the repo
.venvenvironment:.venv/bin/swig -versionreports SWIG 4.4.1..venv/bin/pythonsuccessfully parsespyproject.toml..venv/bin/python -m py_compile setup.pypasses.rst2pseudoxml.git diff --checkpasses..venv/bin/swigsatisfies the new4.4.1...<5requirement.No runtime tests were added because this is a build dependency/configuration update.
Documentation
Added a release-note snippet describing SWIG ABI 5 support between BSK and BSK plugins. Updated the known-issues SWIG 4.4.0 guidance. Removed the stale MuJoCo-specific SWIG dependency note so SWIG requirements remain centralized in the build configuration.
Future work
None.