Skip to content

[ecal,tcp-pubsub] Update to 6.1.1 and 2.0.1 - #54308

Merged
Billy O'Neal (BillyONeal) merged 6 commits into
microsoft:masterfrom
miyanyan:update-ecal-6.1.1
Oct 7, 2026
Merged

Billy O'Neal (BillyONeal) merged 6 commits into
microsoft:masterfrom
miyanyan:update-ecal-6.1.1

Conversation

@miyanyan

@miyanyan miyan (miyanyan) commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Update ecal from 5.13.4#2 to 6.1.1 and its tcp-pubsub dependency from 1.0.3 to 2.0.1. This follows up on #49552 and incorporates the dependency-provider and static-dependency feedback from that PR.

  • Adapt the ecal patches to 6.1.1 and use vcpkg dependencies through a CMake dependency provider.
  • Disable building/running ecal_generate_config; the SDK uses built-in configuration defaults. Disable the pbftags generator and use the generated headers included upstream.
  • Preserve Linux static/shared builds and fix static object inclusion and exported dependencies. Windows ecal remains static, including with the x64-windows triplet.
  • Fix Linux packaging of the versioned ecal_process_stub executable and make protobuf component libraries honor static linkage.

Copilot AI balanced review requested due to automatic review settings October 5, 2026 14:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The newly excluded Android triplets still have stale eCAL failure entries in the CI baseline.

Review effort: Balanced
Findings: 1 Low severity

Open (1)
What changed in this PR

Updates eCAL to 6.1.1 and tcp-pubsub to 2.0.1 while improving vcpkg dependency integration and static packaging.

Changes:

  • Updates port manifests and version metadata.
  • Adds an eCAL CMake dependency provider and disables build-time generators.
  • Refreshes patches for dependency discovery, linkage, exports, and tooling.

Review depth: recipes, manifests, patches, and metadata reviewed; builds were not run.

File Description
versions/​t-/​tcp-pubsub.json Adds tcp-pubsub 2.0.1.
versions/​e-/​ecal.json Adds eCAL 6.1.1.
versions/​baseline.json Updates baseline versions.
ports/​tcp-pubsub/​vcpkg.json Updates the port version.
ports/​tcp-pubsub/​use-ports-for-asio-and-recycle.patch Refreshes external dependency integration.
ports/​tcp-pubsub/​portfile.cmake Updates source and build options.
ports/​tcp-pubsub/​fix-package-config-file.patch Removes an obsolete patch.
ports/​ecal/​vcpkg.json Updates version, support, and dependencies.
ports/​ecal/​portfile.cmake Reworks configuration and packaging.
ports/​ecal/​dependencies.cmake Adds the dependency provider.
ports/​ecal/​0010-disable-build-time-tools.patch Disables build-time generators.
ports/​ecal/​0009-protobuf-6.patch Refreshes Protobuf 6 compatibility.
ports/​ecal/​0008-protobuf-linkage.patch Corrects protobuf library linkage.
ports/​ecal/​0007-allow-static-build-of-core.patch Adds static core support and dependencies.
ports/​ecal/​0006-use-find_dependency-in-cmake-config.patch Updates exported dependency discovery.
ports/​ecal/​0005-remove-install-prefix-macro-value.patch Refreshes relocation handling.
ports/​ecal/​0004-install-cmake-files-to-share.patch Removes an obsolete install-layout patch.
ports/​ecal/​0003-fix-dependencies.patch Updates HDF5 discovery.
ports/​ecal/​0002-fix-build.patch Refreshes build-system fixes.
ports/​ecal/​0001-disable-app-plugins.patch Removes an obsolete application patch.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread ports/ecal/vcpkg.json
"homepage": "https://eclipse-ecal.github.io/ecal/",
"license": "Apache-2.0",
"supports": "!emscripten",
"supports": "!emscripten & !uwp & !android",
@miyanyan
miyan (miyanyan) marked this pull request as ready for review October 7, 2026 04:57

@BillyONeal Billy O'Neal (BillyONeal) 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.

Thanks for the updates!

@BillyONeal
Billy O'Neal (BillyONeal) merged commit 4f74f79 into microsoft:master Oct 7, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants