Upgrade dependencies: Qt 6.9.3, Assimp 6.0.4, Ogre 14.5.2 - #162
Conversation
Upgrade CI dependency versions for all three platforms: - Qt 6.9.1 → 6.9.3 (250+ bug fixes, security updates) - Assimp 6.0.2 → 6.0.4 (FBX animation fixes, glTF skinning export improvements, CVE fixes) - Ogre3D 14.4.1 → 14.5.2 (linear workflow, bone bounding radius fix, entity optimization) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2b39420a14
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ASSIMP_VERSION: '6.0.4' | ||
| ASSIMP_DIR_VERSION: '6.0' | ||
| OGRE_VERSION: '14.4.1' | ||
| OGRE_VERSION: '14.5.2' |
There was a problem hiding this comment.
Updating ASSIMP_VERSION/OGRE_VERSION here does not invalidate any actions/cache keys in this workflow (they are static like ${{ runner.os }}-build-${{ env.cache-name }}), so when a prior cache exists the dependency checkout/build steps are skipped and CI continues using previously cached artifacts. In that cache-hit scenario, this upgrade commit can silently build/test/package against old Assimp/Ogre binaries instead of the new versions, which defeats the purpose of the dependency bump and its security fixes.
Useful? React with 👍 / 👎.
|
1 participant
Summary
Notable Ogre 14.5.x improvements relevant to QtMeshEditor
No breaking API changes — the codebase compiles and all tests pass without modification.
Test plan
🤖 Generated with Claude Code