Skip to content

[fastdds] Update to 3.6.2 - #53298

Merged
Billy O'Neal (BillyONeal) merged 3 commits into
microsoft:masterfrom
SunBlack:fastdds
Aug 10, 2026
Merged

[fastdds] Update to 3.6.2#53298
Billy O'Neal (BillyONeal) merged 3 commits into
microsoft:masterfrom
SunBlack:fastdds

Conversation

@SunBlack

Copy link
Copy Markdown
Contributor
  • Changes comply with the maintainer guide.
  • SHA512s are updated for each updated download.
  • The "supports" clause reflects platforms that may be fixed by this new version, or no changes were necessary.
  • Any fixed CI baseline and CI feature baseline entries are removed from that file, or no entries needed to be changed.
  • All patch files in the port are applied and succeed.
  • The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
  • Exactly one version is added in each modified versions file.

@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 update! GPT 5.6 Sol reports the following:

  • find_package(fastdds CONFIG REQUIRED) fails during generation because the exported fastdds target references nlohmann_json::nlohmann_json, while fastdds-config.cmake does not call find_dependency(nlohmann_json CONFIG REQUIRED).
  • A direct MSVC consumer using only the installed include root and all installed .lib files fails because Fast DDS auto-links libfastdds-3.6.lib/libfastddsd-3.6.lib, but the dynamic package installs fastdds-3.6.lib/fastddsd-3.6.lib. Please disable or correct upstream auto-linking for the vcpkg package without requiring consumer-defined macros.
  • The Windows install leaves fast-discovery-server.exe symlinks targeting versioned executables that the port subsequently moves/removes. The resulting broken links also caused vcpkg's binary-cache archive step to fail. Please remove or relocate both Release and Debug aliases consistently.
  • The package compiles vendored Boost, PEGTL, FileWatch, optionparser, and SQLite code, but declares only Apache-2.0 and installs only the top-level Apache license.
  • vcpkg_copy_tool_dependencies() is deprecated; use vcpkg_copy_tools().
  • ports/fastdds/usage is stored with CRLF line endings; non-patch port files should use LF.

[...]

The custom usage file is necessary. Generated usage incorrectly recommends:
target_link_libraries(main PRIVATE fastdds::optionparser)

Would you consider SunBlack#32 ?

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@BillyONeal
Billy O'Neal (BillyONeal) marked this pull request as draft August 8, 2026 05:57
[fastdds] Fix package integrations and licensing
@SunBlack
SunBlack (SunBlack) marked this pull request as ready for review August 9, 2026 20:54

@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!

@BillyONeal
Billy O'Neal (BillyONeal) merged commit 0828941 into microsoft:master Aug 10, 2026
16 checks passed
@SunBlack
SunBlack (SunBlack) deleted the fastdds branch August 10, 2026 17:27
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.

2 participants