Skip to content

fix(vcpkg): update thread-system overlay port with correct configuration#288

Merged
kcenon merged 3 commits into
mainfrom
feature/282-vcpkg-thread-system-overlay-port
Dec 29, 2025
Merged

fix(vcpkg): update thread-system overlay port with correct configuration#288
kcenon merged 3 commits into
mainfrom
feature/282-vcpkg-thread-system-overlay-port

Conversation

@kcenon

@kcenon kcenon commented Dec 29, 2025

Copy link
Copy Markdown
Owner

Summary

  • Update portfile.cmake with correct thread_system CMake options
  • Fix version from 3.0.0 to 0.3.0 to match actual project version
  • Format vcpkg manifest per vcpkg standard conventions
  • Update related documentation

Changes

portfile.cmake

  • Use specific commit SHA (80242646537034754121657f18649939cecd77c6) instead of VERSION variable
  • Replace generic build options with thread_system-specific options:
    • BUILD_WITH_COMMON_SYSTEM=ON (required dependency)
    • THREAD_BUILD_INTEGRATION_TESTS=OFF
    • BUILD_DOCUMENTATION=OFF
    • THREAD_ENABLE_LOCKFREE_QUEUE=ON
    • THREAD_ENABLE_WORK_STEALING=OFF

vcpkg.json

  • Version corrected: 3.0.0 → 0.3.0
  • Formatted per vcpkg standard (removed redundant port-version, reordered dependencies)

Documentation

  • Updated version references in vcpkg-ports/README.md
  • Updated notes in docs/guides/VCPKG_OVERLAY_PORTS.md

Test Plan

  • Verified port recognition via vcpkg install --dry-run
  • Validated manifest format via vcpkg format-manifest
  • Full build test blocked by upstream CMake issue (see VCPKG_OVERLAY_PORTS.md)

Related Issues

Closes #282
Part of #279

- Update version from 3.0.0 to 0.3.0 to match actual project version
- Use specific commit SHA instead of VERSION variable
- Replace generic build options with thread_system-specific options:
  - DBUILD_WITH_COMMON_SYSTEM=ON (required dependency)
  - DTHREAD_BUILD_INTEGRATION_TESTS=OFF
  - DBUILD_DOCUMENTATION=OFF
  - DTHREAD_ENABLE_LOCKFREE_QUEUE=ON
  - DTHREAD_ENABLE_WORK_STEALING=OFF
- Remove MAYBE_UNUSED_VARIABLES as all options are now properly defined

Refs #282
- Remove redundant port-version field (default value)
- Reorder dependencies per vcpkg convention:
  regular deps -> platform-specific -> host deps

Refs #282
- Update thread-system version from 3.0.0 to 0.3.0 in documentation
- Update notes to reflect new portfile approach:
  - Now uses specific commit SHA for reproducibility
  - Uses project-specific CMake options

Refs #282
@kcenon kcenon self-assigned this Dec 29, 2025
@kcenon kcenon merged commit f2c905d into main Dec 29, 2025
21 checks passed
@kcenon kcenon deleted the feature/282-vcpkg-thread-system-overlay-port branch December 29, 2025 11:34
kcenon added a commit that referenced this pull request Apr 13, 2026
…ion (#288)

* fix(vcpkg): update thread-system port with correct CMake options

- Update version from 3.0.0 to 0.3.0 to match actual project version
- Use specific commit SHA instead of VERSION variable
- Replace generic build options with thread_system-specific options:
  - DBUILD_WITH_COMMON_SYSTEM=ON (required dependency)
  - DTHREAD_BUILD_INTEGRATION_TESTS=OFF
  - DBUILD_DOCUMENTATION=OFF
  - DTHREAD_ENABLE_LOCKFREE_QUEUE=ON
  - DTHREAD_ENABLE_WORK_STEALING=OFF
- Remove MAYBE_UNUSED_VARIABLES as all options are now properly defined

Refs #282

* style(vcpkg): format thread-system manifest with vcpkg standard

- Remove redundant port-version field (default value)
- Reorder dependencies per vcpkg convention:
  regular deps -> platform-specific -> host deps

Refs #282

* docs(vcpkg): update thread-system version and port notes

- Update thread-system version from 3.0.0 to 0.3.0 in documentation
- Update notes to reflect new portfile approach:
  - Now uses specific commit SHA for reproducibility
  - Uses project-specific CMake options

Refs #282
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.

vcpkg: Create overlay port for kcenon-thread-system

1 participant