Skip to content

feat(vcpkg): standardize vcpkg.json following ecosystem conventions#373

Merged
kcenon merged 3 commits into
mainfrom
feature/372-standardize-vcpkg-json
Dec 29, 2025
Merged

feat(vcpkg): standardize vcpkg.json following ecosystem conventions#373
kcenon merged 3 commits into
mainfrom
feature/372-standardize-vcpkg-json

Conversation

@kcenon

@kcenon kcenon commented Dec 29, 2025

Copy link
Copy Markdown
Owner

Summary

Standardize vcpkg.json to follow the unified_system ecosystem conventions established in common_system.

  • Add $schema for IDE validation and autocomplete
  • Rename package from network-system to kcenon-network-system
  • Add version (1.0.0), port-version, homepage, and supports fields
  • Add ecosystem dependencies (common, thread, logger, container systems)
  • Add fmt with version constraint (>=10.0.0)
  • Move gtest and benchmark to testing feature
  • Add ssl feature with OpenSSL version constraint (>=3.0.0)
  • Update README documentation to reflect correct dependencies

Changes

vcpkg.json

  • Package name: network-systemkcenon-network-system
  • Added fields: $schema, version, port-version, homepage, supports
  • Dependencies: Added all 4 ecosystem dependencies + fmt version constraint
  • Features: Moved testing deps to testing feature, added ssl feature

Documentation

  • Updated Dependencies section in README.md and README_KO.md
  • Moved ecosystem dependencies from Optional to Required
  • Added fmt and zlib as required dependencies
  • Updated OpenSSL version recommendation

Test plan

  • JSON syntax validation passed
  • Follows ecosystem template from common_system/thread_system
  • CI workflow validation
  • vcpkg manifest mode build verification (blocked by ecosystem deps)

Resolves #372

- Add $schema for IDE validation and autocomplete
- Rename package from 'network-system' to 'kcenon-network-system'
- Add version (1.0.0), port-version, homepage, and supports fields
- Add ecosystem dependencies (common, thread, logger, container systems)
- Add fmt with version constraint (>=10.0.0)
- Move gtest and benchmark to testing feature
- Add ssl feature with openssl version constraint (>=3.0.0)

Resolves #372
- Move ecosystem dependencies from Optional to Required section
- Add common_system as required dependency
- Update OpenSSL version recommendation (3.x recommended / 1.1.1+ minimum)
- Add fmt (10.0.0+) and zlib as required dependencies
- Update CMake version requirement to 3.20+
- Sync changes to Korean README (README_KO.md)
@github-actions

Copy link
Copy Markdown
Contributor

Performance Comparison

Base Branch Results

No base results

PR Branch Results

No PR results

Ecosystem dependencies (kcenon-common-system, kcenon-thread-system,
kcenon-logger-system, kcenon-container-system) are not registered in
the official vcpkg registry yet. The CI uses direct GitHub checkout
for these dependencies.

These dependencies will be added back once:
- kcenon/common_system#252
- kcenon/thread_system#348
- kcenon/logger_system#264
- kcenon/container_system#202
are resolved and packages are registered in vcpkg registry.

See #371 for tracking ecosystem dependency registration.
@github-actions

Copy link
Copy Markdown
Contributor

Performance Comparison

Base Branch Results

No base results

PR Branch Results

No PR results

@kcenon kcenon merged commit 2b8b8d0 into main Dec 29, 2025
41 checks passed
@kcenon kcenon deleted the feature/372-standardize-vcpkg-json branch December 29, 2025 14:19
kcenon added a commit that referenced this pull request Apr 13, 2026
…373)

* feat(vcpkg): standardize vcpkg.json following ecosystem conventions

- Add $schema for IDE validation and autocomplete
- Rename package from 'network-system' to 'kcenon-network-system'
- Add version (1.0.0), port-version, homepage, and supports fields
- Add ecosystem dependencies (common, thread, logger, container systems)
- Add fmt with version constraint (>=10.0.0)
- Move gtest and benchmark to testing feature
- Add ssl feature with openssl version constraint (>=3.0.0)

Resolves #372

* docs: update dependencies to reflect vcpkg.json changes

- Move ecosystem dependencies from Optional to Required section
- Add common_system as required dependency
- Update OpenSSL version recommendation (3.x recommended / 1.1.1+ minimum)
- Add fmt (10.0.0+) and zlib as required dependencies
- Update CMake version requirement to 3.20+
- Sync changes to Korean README (README_KO.md)

* fix(vcpkg): remove ecosystem dependencies not yet in vcpkg registry

Ecosystem dependencies (kcenon-common-system, kcenon-thread-system,
kcenon-logger-system, kcenon-container-system) are not registered in
the official vcpkg registry yet. The CI uses direct GitHub checkout
for these dependencies.

These dependencies will be added back once:
- kcenon/common_system#252
- kcenon/thread_system#348
- kcenon/logger_system#264
- kcenon/container_system#202
are resolved and packages are registered in vcpkg registry.

See #371 for tracking ecosystem dependency registration.
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.

Standardize vcpkg.json following ecosystem conventions

1 participant