fix(libmoq): declare the full Windows system-lib set for C consumers

[kixelated]

When an external linker consumes libmoq.a on Windows, cargo cannot inject the system libraries Rust std/deps require, so they must be declared in the CMake config consumers link against.

The find_package config (rs/libmoq/cmake/moq-config.cmake.in) listed only ws2_32/userenv/bcrypt/ntdll; this adds dbghelp/advapi32/legacy_stdio_definitions to match cargo rustc -- --print native-static-libs. The same set is mirrored onto the add_subdirectory moq target so both consumption paths (released config + in-tree) link cleanly.

Surfaced while linking libmoq.a into a C++ consumer on Windows (__imp_NtCreateNamedPipeFile and similar unresolved externals).

(Written by Claude)

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4d92ec90-da03-4eac-a350-8b6c3a10d372

📥 Commits

Reviewing files that changed from the base of the PR and between 2caa732 and b47cc06.

📒 Files selected for processing (2)

• rs/libmoq/CMakeLists.txt
• rs/libmoq/cmake/moq-config.cmake.in

---

Walkthrough

Two CMake files for the libmoq Rust static library are updated to declare Windows-specific interface link libraries. CMakeLists.txt gains a WIN32 conditional block on the moq INTERFACE target's target_link_libraries call, adding ntdll, userenv, ws2_32, bcrypt, dbghelp, advapi32, and legacy_stdio_definitions. The installed config template moq-config.cmake.in mirrors this change by expanding the INTERFACE_LINK_LIBRARIES property in its own WIN32 block with the same additional libraries, reformatted across multiple lines.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the main change: declaring the full set of Windows system libraries needed for C consumers of libmoq. It is concise, specific, and directly related to the primary change in the changeset.
Description check	✅ Passed	The description provides clear context about the Windows linking issue, explains why the change is needed, details what was modified, and references the cargo command used to determine the correct libraries. It is directly related to and explains the changeset.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

✨ Simplify code

• Create PR with simplified code
• Commit simplified code in branch claude/libmoq-windows-link-libs

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}