Skip to content

[fancyargumentparser] Add new port at version 1.0.1 - #52909

Merged
Victor Romero (vicroms) merged 6 commits into
microsoft:masterfrom
simfeo:add-fancyargumentparser
Jul 16, 2026
Merged

[fancyargumentparser] Add new port at version 1.0.1#52909
Victor Romero (vicroms) merged 6 commits into
microsoft:masterfrom
simfeo:add-fancyargumentparser

Conversation

@simfeo

@simfeo simfeo commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Description

This PR adds a new header-only port for fancyargumentparser.

New Port Checklist

  • Changes comply with the maintainer guide.
  • The packaged project is mature and ready for broad sharing with vcpkg users.
  • The packaged project shows strong association with the chosen port name.
  • The versioning scheme in vcpkg.json matches what upstream says.
  • The license declaration in vcpkg.json matches what upstream says.
  • The installed file as the "copyright" file matches what upstream says.
  • The source code of the component installed comes from an authoritative source.
  • 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.

@simfeo

simfeo commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

Comment thread ports/fancyargumentparser/portfile.cmake Outdated
Comment on lines +12 to +13
set(VCPKG_POLICY_EMPTY_INCLUDE_FOLDER enabled)
set(VCPKG_POLICY_DLLS_IN_STATIC_LIBRARY enabled)

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.

Suggested change
set(VCPKG_POLICY_EMPTY_INCLUDE_FOLDER enabled)
set(VCPKG_POLICY_DLLS_IN_STATIC_LIBRARY enabled)
set(VCPKG_POLICY_EMPTY_INCLUDE_FOLDER enabled)
set(VCPKG_POLICY_DLLS_IN_STATIC_LIBRARY enabled)

It this is a header-only port, there shouldn't be an empty include folder also no DLLs should be produced to require the second policy.

Comment thread ports/fancyargumentparser/portfile.cmake
@vicroms
Victor Romero (vicroms) marked this pull request as draft July 15, 2026 20:06
@simfeo
simfeo marked this pull request as ready for review July 15, 2026 20:21
@vicroms
Victor Romero (vicroms) marked this pull request as draft July 15, 2026 21:22
@simfeo
simfeo marked this pull request as ready for review July 16, 2026 11:22
@vicroms
Victor Romero (vicroms) merged commit ec62869 into microsoft:master Jul 16, 2026
16 checks passed
Yury Bura (yurybura) pushed a commit to yurybura/vcpkg that referenced this pull request Jul 17, 2026
Co-authored-by: Victor Romero <romerosanchezv@gmail.com>
@simfeo
simfeo deleted the add-fancyargumentparser branch July 23, 2026 20:03
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